No description provided.
Trivial: grammar fix to CONTRIBUTING.md #10513
pull ABISprotocol wants to merge 1 commits into bitcoin:master from ABISprotocol:patch-1 changing 1 files +2 −2-
ABISprotocol commented at 10:15 PM on June 2, 2017: none
-
Trivial: grammar fix to CONTRIBUTING.md 16cc466af1
- fanquake added the label Docs and Output on Jun 2, 2017
-
paveljanik commented at 7:06 AM on June 3, 2017: contributor
If
"pull requests."is good grammar, the grammar is bad.NACK
-
fanquake commented at 1:36 AM on June 5, 2017: member
Agree with @paveljanik, neither change seems to be fixing any grammar.
- fanquake closed this on Jun 5, 2017
-
ABISprotocol commented at 1:42 AM on June 6, 2017: none
Although this is marked Trivial, I request that you reopen this because of the following reasons.
Alternatively, please retract statements you have made that "grammar is bad" - @paveljanik - and that "neither change seems to be fixing any grammar" - @fanquake - and then propose a new issue dealing with the grammar issues which come up when dealing with the different ways that different countries handle periods (as in some countries the period concluding the sentence, for example, is placed inside the quotation marks, and in others place it is placed outside the quotation marks).
However, it would be improper to close this issue due to your claims that "the grammar is bad" or that "neither change seems to be fixing any grammar."
As an example... this merged pull request -- https://github.com/bitcoin/bitcoin/pull/6962/commits/6dd3a44ce2cb4480054ae0ca0960f588a9ad8d25 -- was provided so as to not translate markdown or force English grammar, but it certainly did not clarify what to do about differences in English grammar with respect to how people in different countries handle commas and periods with respect to quotation marks (in determining whether to place them inside or outside quotation marks).
Suggested solution:
Reopen this issue, or open a new issue to discuss further the inconsistency identified above. @laanwj
Note (Edited): It was my intention with this to allow edits from maintainers, so that users with write access to bitcoin/bitcoin would be able to add new commits to my patch-1 branch for this. However, something about this setting apparently is problematic and doesn't want to stay; it is reverting to not allow edits from maintainers for some reason.
-
paveljanik commented at 5:33 AM on June 6, 2017: contributor
I can't retract my opinion. I'm non English native speaker, do not care about en_US and en_UK differences and do not care about your side in that "fight".
-
ABISprotocol commented at 6:10 AM on June 6, 2017: none
@paveljanik It's an interesting position, not to care about someone else's thoughts or views. It's not a fight, but it is a legitimate question. As said, trivial, but look at it like this. What logic is there in imposing UK grammar on everybody who is using English?
It strikes me that (again, as pointed out) that although this is trivial, it is worth a bit of discussion.
-
laanwj commented at 8:56 AM on June 6, 2017: member
Please don't tag me in this, I have no duck in this fight. Life is too short to have long arguments about minor language issues IMO. I try to focus on changes affecting performance, user experience, stability and testing.
- ABISprotocol deleted the branch on Jun 6, 2017
-
ABISprotocol commented at 3:48 PM on June 6, 2017: none
Not intended to be a fight, as mentioned, but if you are focusing on changes affecting user experience, this would be one of them.
- MarcoFalke locked this on Sep 8, 2021