Skip to content

Resolving grammatical issue in README.md file #314

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Aug 18, 2016
Merged

Resolving grammatical issue in README.md file #314

merged 2 commits into from
Aug 18, 2016

Conversation

mrsteele
Copy link
Contributor

@mrsteele mrsteele commented Aug 5, 2016

Just spotted this light grammatical error and thought I would resolve it.

Ended up changing the whole sentence as it didn't seem quite right, but please let me know if you think it is worse by... well... denying the pull 😉

Updating a sentence in the README.md file due to grammatical error and didn't add any value to the user.
Resolving an issue introduced in previous commit.
@codecov-io
Copy link

codecov-io commented Aug 5, 2016

Current coverage is 98.21% (diff: 100%)

Merging #314 into master will not change coverage

@@             master       #314   diff @@
==========================================
  Files             9          9          
  Lines           280        280          
  Methods          39         39          
  Messages          0          0          
  Branches         60         60          
==========================================
  Hits            275        275          
  Misses            5          5          
  Partials          0          0          

Powered by Codecov. Last update 9d2941b...c2e5a9e

@bebraw
Copy link
Contributor

bebraw commented Aug 14, 2016

Looks good to me. 👍

@mrsteele
Copy link
Contributor Author

@bebraw Would you mind merging this in please?

@bebraw
Copy link
Contributor

bebraw commented Aug 16, 2016

I think the maintainers will get onto it eventually. 👍

@sokra sokra merged commit 068cec3 into webpack-contrib:master Aug 18, 2016
@sokra
Copy link
Member

sokra commented Aug 18, 2016

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants