<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Nov 11, 2013 at 1:28 PM, Tim Bell <span dir="ltr"><<a href="mailto:Tim.Bell@cern.ch" target="_blank">Tim.Bell@cern.ch</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
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>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>Agreed on suggesting and even patching the patch to submit the replacement text. Spelling and grammar count.</div><div><br></div>
<div>Related to the v3 Nova API - I would also like to see a wiki page that outlines our preferred wording and spelling so that reviewers have a "final say" wiki page to refer to. For example, I heard we want to move towards "servers" rather than "instances" for the API, but I'd need that written down somewhere to be sure I'm recalling it correctly. Does anyone have interest on these lists? If not I'll ask a docs person to start one.</div>
<div><br></div><div>At docs boot camp, we talked about English-second-language coders and reviewers and I'm requesting here that the native speakers step up to write things down. I hadn't realized how many non-native speakers we have, it's awesome, so let's accommodate while still valuing good communication.</div>
<div><br></div><div>Thanks,</div><div>Anne</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888">
Tim<br>
</font></span><div class="im HOEnZb"><br>
> -----Original Message-----<br>
> From: John Griffith [mailto:<a href="mailto:john.griffith@solidfire.com">john.griffith@solidfire.com</a>]<br>
> Sent: 11 November 2013 20:03<br>
> To: OpenStack Development Mailing List (not for usage questions)<br>
> Subject: Re: [openstack-dev] [qa] Policy on spelling and grammar<br>
><br>
</div><div class="HOEnZb"><div class="h5">> On Tue, Nov 12, 2013 at 2:49 AM, James Slagle <<a href="mailto:james.slagle@gmail.com">james.slagle@gmail.com</a>> wrote:<br>
> > -1 from me as well.<br>
> ><br>
> > When I first started with OpenStack, I probably would have agreed with<br>
> > letting small grammar mistakes and typos slide by.<br>
> ><br>
> > However, I now feel that getting commit messages right is more<br>
> > important. Also keep in mind that with small grammar mistakes, the<br>
> > intent may be obvious to a native English speaker, but to another<br>
> > non-native English speaker it may not be. And just a few small<br>
> > grammar mistakes/misspellings/typos can add up until the meaning may<br>
> > be harder to figure out for another non-native English speaker.<br>
> ><br>
> > Also, I can't speak for everyone, but in general I've found most folks<br>
> > open to grammar corrections if English is not their native language<br>
> > b/c they want to learn and fix the mistakes.<br>
> ><br>
> ><br>
> > --<br>
> > -- James Slagle<br>
> > --<br>
> ><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>
> Guess I'm in the minority with here... some of the nits in commit messages and comments is a bit extreme. Sure there are some cases<br>
> where I think offering a correction is great/appropriate, but for example issuing a -1 on somebody's patch because they mixed up their<br>
> use of 'there' seems a bit lame.<br>
><br>
> Seems to me there's a middle ground here, but honestly if you're value add to the review process is catching grammatical or spelling<br>
> errors in comments and commit messages I'd argue that in most cases it would be nice to have more substantive feedback to go along<br>
> with it. I happen to be a top offender here in terms of grammar or spelling errors in comments so I'm a bit biased on the topic. :)<br>
><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>
_______________________________________________<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>
</div></div></blockquote></div><br></div></div>