<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Nov 12, 2013 at 6:24 AM, Ben Nemec <span dir="ltr"><<a href="mailto:openstack@nemebean.com" target="_blank">openstack@nemebean.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 2013-11-11 13:28, Tim Bell wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
As a speaker of the Queen's English, I find flavor to be incorrect.<br>
Does that mean I can -1 any patch that does not use flavour ?<br>
<br>
At CERN, we are working with 130 countries in a single community. The<br>
value of the contribution of non-english speakers far exceeds the<br>
occasional misunderstandings.<br>
<br>
Giving grammar/spellings -1 excludes major sections of the community<br>
from contribution.<br>
<br>
As our aim is meritocracy (in python, computer architecture and design<br>
rather than spelling), I'd propose<br>
<br>
- If someone identifies a need for clarification/correction as part of<br>
a review, they also submit the replacement text rather than just -1.<br>
- The submitter incorporates that change into a patch<br>
</blockquote>
<br></div>
Agreed.  Whenever possible, a grammar/spelling -1 should include proposed alternatives (I say whenever possible because I've -1'd a few changes where the docstrings were incomprehensible to me, so I couldn't provide a better alternative as I wasn't sure what they were trying to say :-).<span class="HOEnZb"><font color="#888888"><br>

<br></font></span></blockquote><div><br></div><div>I'm much more likely to -1 spelling/grammar issues in the code base than in a commit message. For both cases I agree its important to provide exactly what you want because for a non native english speaker is likely to come up<br>
with many variants which aren't right even if it seems obvious to a native english speaker and there is no easy way for them to check it before resubmitting.<br></div><div><br></div><div>For commit messages I've been more lenient on new contributors and changesets that have already gone through very many iterations as long as it is reasonably understandable by a native english speaker, though as someone else has mentioned in this thread perhaps we do need to consider how confusing it might be for non english speakers. <br>
<br>I don't think we should be -1'ing for punctuation, and I've +1/+2'd before for minor grammar or spelling issues in commit messages, with just a note to fix it if they have to update the patch anyway. The main reason I really dislike +0'ing anything with just a comment is that Gerritt's review request list continues to show that I haven't reviewed it which I find pretty annoying as I keep going back to it.<br>
</div><div><br></div>Chris<br><br></div></div></div>