<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 12 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:Tahoma;
        panose-1:2 11 6 4 3 5 4 4 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:12.0pt;
        font-family:"Times New Roman","serif";}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:purple;
        text-decoration:underline;}
span.hoenzb
        {mso-style-name:hoenzb;}
span.EmailStyle18
        {mso-style-type:personal-reply;
        font-family:"Calibri","sans-serif";
        color:#1F497D;}
.MsoChpDefault
        {mso-style-type:export-only;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal" style="margin-left:.5in"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> Christopher Yeoh [mailto:cbkyeoh@gmail.com]
<br>
<b>Sent:</b> Monday, November 11, 2013 3:06 PM<br>
<b>To:</b> openstack@nemebean.com; OpenStack Development Mailing List (not for usage questions)<br>
<b>Subject:</b> Re: [openstack-dev] [qa] Policy on spelling and grammar<o:p></o:p></span></p>
</div>
<p class="MsoNormal" style="margin-left:.5in"><o:p> </o:p></p>
<div>
<div>
<div>
<p class="MsoNormal" style="margin-left:.5in">On Tue, Nov 12, 2013 at 6:24 AM, Ben Nemec <<a href="mailto:openstack@nemebean.com" target="_blank">openstack@nemebean.com</a>> wrote:<o:p></o:p></p>
<div>
<p class="MsoNormal" style="margin-left:.5in">On 2013-11-11 13:28, Tim Bell wrote:<o:p></o:p></p>
<p class="MsoNormal" style="margin-left:.5in">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<o:p></o:p></p>
<p class="MsoNormal" style="margin-left:.5in"><o:p> </o:p></p>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:0in;margin-right:0in;margin-bottom:12.0pt;margin-left:.5in">
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 :-).<o:p></o:p></p>
<div>
<p class="MsoNormal" style="margin-left:.5in"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in">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.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in">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.<o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Totally agree with chris on this.  The +0s tend seem to fall into some weird void.  They exist, but not on the same plane as the rest of the +/-  Perhaps zero
 had not been invented when Gerrit was first created ;-)<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">--Rocky<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in"><o:p> </o:p></p>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:0in;margin-right:0in;margin-bottom:12.0pt;margin-left:.5in">
Chris<o:p></o:p></p>
</div>
</div>
</div>
</div>
</body>
</html>