Update for active voice, clarity and grammar #457
Conversation
Compiler configuration section not included because of open PR ARMmbed#453
|
@AnotherButler could you please confirm when you're ready with the changes/fixes? |
README.md
Outdated
|
|
||
| ``` | ||
| $ git remote set-url --push origin https://github.com/screamerbg/repo-fork | ||
| ``` | ||
|
|
||
| Each time you `git` commit and push, or use `mbed publish`, the new revisions will be pushed against your fork. You can fetch from the original repository using `mbed update` or `git pull`. If you explicitly want to fetch or pull from your fork, then you can use `git pull https://github.com/screamerbg/repo-fork [branch]`. | ||
| Each of `git commit`, `git push` and `mbed publish` push the new revisions to your fork. You can fetch from the original repository using `mbed update` or `git pull`. If you explicitly want to fetch or pull from your fork, then you can use `git pull https://github.com/screamerbg/repo-fork [branch]`. |
There was a problem hiding this comment.
This isn't quite right. git commit and git push are the equivalent of mbed publish. So they can't be flattened to a list. The two git commands must remain paired.
So perhaps
Both
git commit & git pushandmbed publishpush the new revisions...
The & is valid command syntax, I'm not using it as an English shortcut :)
|
|
||
| To update your program to another upstream version, go to the root folder of the program and run: | ||
| To update your program to another upstream version, go to the root folder of the program, and run: |
There was a problem hiding this comment.
Oxford comma? Or is this a corner case 😄 (I am actually curious though!)
There was a problem hiding this comment.
Haha, no. When a coordinating conjunction joins two independent clauses, you must add a comma before the conjunction.
Make suggested edit about git commit and git push.
|
@screamerbg and @bridadan I think this is ready now. Please confirm. |
|
Thanks @AnotherButler |
Compiler configuration section not included because of open PR #453