Skip to content
This repository has been archived by the owner on Jan 23, 2022. It is now read-only.

chore(readme): fixed typos and grammar #36

Merged
merged 1 commit into from
Sep 13, 2013
Merged

chore(readme): fixed typos and grammar #36

merged 1 commit into from
Sep 13, 2013

Conversation

btesser
Copy link
Contributor

@btesser btesser commented Sep 12, 2013

Fixed typos and grammatical errors in Readme.md

@m7r
Copy link
Owner

m7r commented Sep 13, 2013

Thanks so much for the corrections. I am not a native speaker and its welcome if someone read over my stuff.

Could you change the commit message please.
Something like chore(readme): fixed typos and grammar

@btesser
Copy link
Contributor Author

btesser commented Sep 13, 2013

No problem. I'll take care of that this morning
On Sep 13, 2013 4:42 AM, "m7r" [email protected] wrote:

Thanks so much for the corrections. I am not a native speaker and its
welcome if someone read over my stuff.

Could you change the commit message please.
Something like chore(readme): fixed typos and grammar


Reply to this email directly or view it on GitHubhttps://github.com//pull/36#issuecomment-24380809
.

@btesser
Copy link
Contributor Author

btesser commented Sep 13, 2013

Just clarifying, do you mean modify the commit or the pull request title? Not quite sure how to modify commit title offhand

@m7r
Copy link
Owner

m7r commented Sep 13, 2013

The commit title.
These directions are copied from angularjs:
If you need to make changes to your pull request, you can update the commit with git commit --amend.
Then, update the pull request with git push -f.

Fixed spelling and grammatical errors in Readme.md
@btesser
Copy link
Contributor Author

btesser commented Sep 13, 2013

Ok it should be good now

m7r pushed a commit that referenced this pull request Sep 13, 2013
chore(readme): fixed typos and grammar
@m7r m7r merged commit 5e914be into m7r:master Sep 13, 2013
@m7r
Copy link
Owner

m7r commented Sep 13, 2013

Thanks for you help

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

Successfully merging this pull request may close these issues.

2 participants