Skip to content
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

Correct minor grammar issue #4

Merged
merged 2 commits into from
Jul 3, 2015
Merged

Correct minor grammar issue #4

merged 2 commits into from
Jul 3, 2015

Conversation

scw
Copy link
Contributor

@scw scw commented Jul 3, 2015

'Advice' is a recommendation, 'advise' is a verb meaning 'to give advice', looks good other than in these two spots. Can also be corrected in project description field.

cohenjo added a commit that referenced this pull request Jul 3, 2015
Correct minor grammar issue
@cohenjo cohenjo merged commit c33e66a into cohenjo:master Jul 3, 2015
@cohenjo
Copy link
Owner

cohenjo commented Jul 3, 2015

Hi,
Thanks for the correction - learning something new every day :)
I'm not a native English speaker, so I hope you'll forgive me :p

Regards,

  • Jony

@scw
Copy link
Contributor Author

scw commented Jul 4, 2015

Jony,

No problem at all. Just gave a brief explanation because it's really not obvious and I didn't catch it myself till rereading the docs. Thanks for the quick pull.

Cheers,
Shaun

@scw scw deleted the scw-patch-1 branch July 4, 2015 04:33
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.

2 participants