<div dir="ltr"><div class="gmail_extra">On Mon, Nov 11, 2013 at 12:19 PM, Clint Byrum <span dir="ltr"><<a href="mailto:clint@fewbar.com" target="_blank">clint@fewbar.com</a>></span> wrote:<br><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Excerpts from David Kranz's message of 2013-11-11 09:58:59 -0800:<br>
<div class="im">> I have seen a wide variety of -1'ing (and in many cases approving)<br>
> patches for minor spelling or grammatical errors and think we need a<br>
> policy about this. Given the large number of contributors for whom<br>
> English is not their native language, I would be in favor of rejecting<br>
> spelling errors in variable or method names but being more lenient in<br>
> comments, commit messages, READMEs, etc. What do you all think?<br>
><br>
<br>
</div>The point of code review is to find defects. Misspelled words are defects<br>
in the English language encoded in the change. In fact, commit messages<br>
in particular are critical to get right as they cannot ever be fixed,<br>
and they are generally the most useful when under a stressful situation<br>
trying to determine the nature of a regression.<br>
<br>
Many of our contributors are also newbies to python, and we do not let<br>
them get away with obvious mistakes in python code. English is just a<br>
language with a different interpreter (a more forgiving one, for sure,<br>
but also one with many versions at various stages of implementation).<br>
<br>
In fact, our large percentage of non-native english speakers is a reason<br>
to be extremely careful about grammar and spelling so as not to confuse<br>
non-native speakers with incorrect grammar and spelling.<br>
<br>
I believe that if a -1 for a spelling mistake is causing more than an<br>
extremely short turn around time then either the submitter is not engaged<br>
with the project and thus not responsive to the -1, or the reviewers<br>
are over-taxed and the given project needs more reviewers.<br>
<div class="HOEnZb"><div class="h5"><br>
</div></div></blockquote><div><br></div><div><br></div><div>It would be so much nicer if there were some easy way for the reviewer himself to fix the typos directly (in a way that can trivially be accepted by the submitter of the patch into his own patch -- with a click of a button).</div>
<div><br></div><div><br></div></div>-- <br>Christopher Armstrong<br><a href="http://radix.twistedmatrix.com/" target="_blank">http://radix.twistedmatrix.com/</a><br><a href="http://planet-if.com/" target="_blank">http://planet-if.com/</a><br>
<br>
</div></div>