Re: [PATCH] documentation: send-email: use camel case consistently

Dragan Simic <dsimic@manjaro.org> writes:

I do not insist; my job is to just reject what is not correct.

In this particular case, I do not think Cc is outright wrong; it is near the borderline, but I do not know which side of that line it sits.

I gave you one possible rule to decide what to capitalize (namely, acronyms are spelled in all caps and that is how we capitalize http.proxySSLCert and imap.preformattedHTML) and if we adopt that rule, then sendemail.supressCc would be incorrect, simply because carbon-copy should be spelled CC.

You need to give an alternative criteria that is easy to understand for future developers and follow, and explain your choice in the proposed commit log message: "We spell acronyms in all caps like HTML and SSL, but in the case of carbon-copy, we spell it as Cc because ...".

You need to fill that "..." is in your proposed log message to explain the choice you made in your patch text. More importantly, it is to help future developers so that they can easily follow the same rule to spell the variable names they invented in a way consistent with the rule you followed in this patch.

Thanks.