<font size=2 face="sans-serif">Since I am one of those people that comments
on spelling and grammar ... If you use emacs for your coding you can add
the following to help catch spelling issues:  </font><a href=http://www.emacswiki.org/emacs/FlySpell><font size=2 face="sans-serif">http://www.emacswiki.org/emacs/FlySpell</font></a>
<br>
<br><font size=2 face="sans-serif">There is also git-commit-mode:  </font><a href=http://www.emacswiki.org/emacs/GitCommitMode><font size=2 face="sans-serif">http://www.emacswiki.org/emacs/GitCommitMode</font></a><font size=2 face="sans-serif">
which ensures good commit message structure.  When combined with FlySpell
many of the simple mistakes can be avoided.<br>
</font>
<br><font size=3><b><i><br>
Jay S. Bryant</i></b><br>
       <i>IBM Cinder Subject Matter Expert 
&  Cinder Core Member</i><br>
                  </font><img src=cid:_2_0C6B73000BDA739C0071D89A86257C36><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">David Kranz <dkranz@redhat.com></font>
<br><font size=1 color=#5f5f5f face="sans-serif">To:      
 </font><font size=1 face="sans-serif">openstack-dev@lists.openstack.org,
</font>
<br><font size=1 color=#5f5f5f face="sans-serif">Date:      
 </font><font size=1 face="sans-serif">12/03/2013 01:50 PM</font>
<br><font size=1 color=#5f5f5f face="sans-serif">Subject:    
   </font><font size=1 face="sans-serif">Re: [openstack-dev]
Tool for detecting commonly misspelled words</font>
<br>
<hr noshade>
<br>
<br>
<br><tt><font size=2>On 12/03/2013 02:05 PM, John Griffith wrote:<br>
> On Tue, Dec 3, 2013 at 11:54 AM, Nachi Ueno <nachi@ntti3.com>
wrote:<br>
>> 2013/12/3 John Griffith <john.griffith@solidfire.com>:<br>
>>> On Tue, Dec 3, 2013 at 11:38 AM, Russell Bryant <rbryant@redhat.com>
wrote:<br>
>>>> On 12/03/2013 09:22 AM, Joe Gordon wrote:<br>
>>>>> HI all,<br>
>>>>><br>
>>>>> Recently I have seen a few patches fixing a few typos.
 I would like to<br>
>>>>> point out a really nifty tool to detect commonly misspelled
words.  So<br>
>>>>> next time you want to fix a typo, instead of just
fixing a single one<br>
>>>>> you can go ahead and fix a whole bunch.<br>
>>>>><br>
>>>>> </font></tt><a href="https://github.com/lyda/misspell-check"><tt><font size=2>https://github.com/lyda/misspell-check</font></tt></a><tt><font size=2><br>
>>>>><br>
>>>>> To install it:<br>
>>>>>    $ pip install misspellings<br>
>>>>><br>
>>>>> To use it in your favorite openstack repo:<br>
>>>>>   $ git ls-files | grep -v locale | misspellings
-f -<br>
>>>>><br>
>>>>><br>
>>>>> Sample output:<br>
>>>>><br>
>>>>> </font></tt><a href=http://paste.openstack.org/show/54354><tt><font size=2>http://paste.openstack.org/show/54354</font></tt></a><tt><font size=2><br>
>>>> Are we going to start gating on spellcheck of code and
commit messages?  :-)<br>
>>> NO please (please please please).  We have enough "grammar
reviewers"<br>
>>> at this point already IMO and I honestly think I might puke
if jenkins<br>
>>> fails my patch because I didn't put a '.' at the end of my
comment<br>
>>> line in the code.  I'd much rather see us focus on things
like... I<br>
>>> dunno... maybe having the code actually work?<br>
>> yeah, but may be non-voting reviews by this tool is helpful<br>
> Fair enough... don't get me wrong I'm all for support of non-english<br>
> contributors etc.  I just think that the emphasis on grammar
and<br>
> punctuation in reviews has gotten a bit out of hand as of late.  FWIW<br>
> I've never -1'd a patch (and never would) because somebody used "its"<br>
> rather than "it's" in a comment.  Or they didn't end
a comment (NOT a<br>
> docstring) with a period.  I think it's the wrong place to spend<br>
> effort quite honestly.<br>
><br>
> That being said, I realize people will continue to this sort of thing<br>
> (it's very important to get your -1 counts in the review stats) and<br>
> admittedly there is some value to spelling and grammar.  I just
feel<br>
> that there are *real* issues and bugs that people could spend this<br>
> time that would actually have some significant and real benefit.<br>
><br>
> I'm obviously in the minority on this topic so I should probably just<br>
> yield at this point and get on board the grammar train.<br>
><br>
><br>
I agree with you. But the last thread about this proved there is no <br>
consensus. The beauty of a tool like this, run by individuals before <br>
they submit, is that it will to some degree make this contentious issue
<br>
moot. I always run a spell checker on my own documents and I'm sure it
<br>
will give more confidence to non-native English speakers to run this on
<br>
their patches. So next time I see a misspelling in a review, I will <br>
simply point the author at this tool to use in the future.<br>
<br>
  -David<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>