Re: [PATCH 1/2] doc: fix small issues in SubmittingPatches

On Tue, Aug 1, 2017 at 8:59 AM, Kaartic Sivaraam
[off-list ref] wrote:

On Mon, 2017-07-31 at 13:23 -0700, Stefan Beller wrote:

quoted

On Sun, Jul 30, 2017 at 9:18 AM, Kaartic Sivaraam
[off-list ref] wrote:

quoted

Replace the dashed version of a command with undashed
version and quote it.

I like it, but similar as below, we'd want to go for
consistency.

I assume you mean the consistency in quoting i.e., you're expecting the
patch to use (") instead of (') for quoting. Correct me, if I'm wrong.

Actually I did not imply any expectation on the outcome, because I
do not know what the consistency end game looks like for this issue.

So maybe we'd want to go with the currently most used way?
(Are there only three? (a) with dash, (b) with single quotes and
(c) with double quotes?)

quoted

quoted

Further, remove duplicated space character.

https://en.wikipedia.org/wiki/Sentence_spacing
seems like a globally controversial thing. (I assumed
it was some sort of local dialect before researching
it properly)

Man, I had no idea about people using two spaces after period.

quoted

I personally do not mind one way or another regarding
(double) spaces after a period, but I would think we'd
strive for consistency throughout the project.

Consistency is crucial. I'll just remove that part of the patch.

--
Kaartic