<font size=2 face="sans-serif">+2 to Clint's comments.</font>
<br>
<br><font size=2 face="sans-serif">As OpenStack gains a wider following
in the corporate world and given that the code for OpenStack may be seen
by anyone, I think it is very important to get spelling and grammar right.
 I also think it is important to ensure that the commit messages concisely
communicate the change that is being made.</font>
<br>
<br><font size=2 face="sans-serif">I regularly comment on misspellings
and in the case that a commit message is difficult to understand provide
suggestions for rewording to make sure I understand what is being communicated.</font>
<br>
<br><font size=2 face="sans-serif">So, I think -1's are fine as long is
assistance is being provided.</font>
<br>
<br><font size=2 face="sans-serif">The point of OpenStack is to have a
collaborative community.  Some people bring better spelling and grammar
to the table than others as a contribution.<br>
</font>
<br><font size=3><b><i><br>
<br>
Jay S. Bryant</i></b><i><br>
Linux Developer - <br>
    OpenStack Enterprise Edition</i><br>
                   </font><img src=cid:_2_0A6D42C80A6D3EA0006766EF86257C20 alt="OpenStack Community Logo"><font size=3><br>
Department 7YLA, Building 015-2, Office E125, Rochester, MN<br>
Telephone: (507) 253-4270, FAX (507) 253-6410<br>
TIE Line: 553-4270<br>
E-Mail:  jsbryant@us.ibm.com<br>
--------------------------------------------------------------------<br>
 All the world's a stage and most of us are desperately unrehearsed.<br>
                   -- Sean
O'Casey<br>
--------------------------------------------------------------------</font>
<br>
<br>
<br>
<br><font size=1 color=#5f5f5f face="sans-serif">From:      
 </font><font size=1 face="sans-serif">Clint Byrum <clint@fewbar.com></font>
<br><font size=1 color=#5f5f5f face="sans-serif">To:      
 </font><font size=1 face="sans-serif">openstack-dev <openstack-dev@lists.openstack.org>,
</font>
<br><font size=1 color=#5f5f5f face="sans-serif">Date:      
 </font><font size=1 face="sans-serif">11/11/2013 12:21 PM</font>
<br><font size=1 color=#5f5f5f face="sans-serif">Subject:    
   </font><font size=1 face="sans-serif">Re: [openstack-dev]
[qa] Policy on spelling and grammar</font>
<br>
<hr noshade>
<br>
<br>
<br><tt><font size=2>Excerpts from David Kranz's message of 2013-11-11
09:58:59 -0800:<br>
> 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>
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>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
OpenStack-dev@lists.openstack.org<br>
</font></tt><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev"><tt><font size=2>http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</font></tt></a><tt><font size=2><br>
<br>
</font></tt>
<br>