<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Nov 12, 2013 at 6:19 AM, Sean Dague <span dir="ltr"><<a href="mailto:sean@dague.net" target="_blank">sean@dague.net</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">On 11/11/2013 02:28 PM, Tim Bell wrote:<br>


><br>
</div><div class="im">> As a speaker of the Queen's English, I find flavor to be incorrect. 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 value of the contribution of non-english speakers far exceeds the occasional misunderstandings.<br>
><br>
> Giving grammar/spellings -1 excludes major sections of the community from contribution.<br>
><br>
> As our aim is meritocracy (in python, computer architecture and design rather than spelling), I'd propose<br>
><br>
> - If someone identifies a need for clarification/correction as part of a review, they also submit the replacement text rather than just -1.<br>
> - The submitter incorporates that change into a patch<br></div></blockquote><div><br></div><div>When it comes to commit messages, I generally will -1 for any spelling mistake or confusing phrasing in the first line of the commit message, but will let a typo that doesn't hurt the readability of the commit message slide in the remainder of the commit message (while pointing it out and mentioning to fix it if a respin is required).  And will always -1 a patch if the phrasing is so poor that I cannot understand the commit message.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">
<br>
</div>Agreed. If anyone -1s a patch for English, it better have a complete<br>
word for word set of replacement text as part of that review.<br>
<br>
Also, grammar eventually becomes the eye of the beholder, and personal<br>
preference, and regional difference, and style, and.... there are a lot<br>
of variables here. The heated debate over whether or not a period ends a<br>
commit subject shows how gray that is (I honestly only hold firm to<br>
keeping that no-period rule in hacking so people would stop -1ing over<br>
it, because there were actually opposing -1 wars over adding / removing<br>
that period).<br>
<br>
So unless it's actually getting in the way of the contribution being<br>
understood in the future, I'd much rather people leave '0' scored<br>
comments with the grammar / spelling micro-nits.<br>
<br>
There is a real reason for that, many of us with a lot of reviews<br>
completely purge anything with a -1. If you score things with a '0'<br>
review, core reviewers will still look at the code. But I'd hate to have<br>
this giant gauntlet of grammar before the code is getting looked at by<br>
+2ers. That seems a pretty high discouragement to new non native English<br>
speakers.<br>
<span class=""><font color="#888888"><br>
        -Sean<br>
<br>
--<br>
Sean Dague<br>
<a href="http://dague.net" target="_blank">http://dague.net</a><br>
<br>
</font></span><br>_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br></div></div>