-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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
Standardized punctuation. #14186
Standardized punctuation. #14186
Conversation
can you add a |
What difference in effect does this make? |
@arthurnn sorry, I'm new to opensource, is this a correct way to amend the commit message? |
@pmq20 hi, no difference in code, but it might be a little difference when we trying to do searching, or reading... |
I think you should git commit with --amend arg and then git push with the --force arg. Otherwise you won't override the message of the original commit and rather made another one. |
@ch33hau Great spirit and also a great way to start contributing to the community. +1 😄 |
@pmq20 thanks...do I need to remove my second and third commits? I'm finding a way to do it... |
@ch33hau You should squash all your commits in one, and force push. That commit must have a more on squashing commits http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html |
Punctuation are not standardized, for same purpose, some use single quotation, some use double quotation: