<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 3, 2013 at 12:46 PM, John Griffith <span dir="ltr"><<a href="mailto:john.griffith@solidfire.com" target="_blank">john.griffith@solidfire.com</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 Tue, Dec 3, 2013 at 11:38 AM, Russell Bryant <<a href="mailto:rbryant@redhat.com">rbryant@redhat.com</a>> 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>
>> <a href="https://github.com/lyda/misspell-check" target="_blank">https://github.com/lyda/misspell-check</a><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>
>> <a href="http://paste.openstack.org/show/54354" target="_blank">http://paste.openstack.org/show/54354</a><br>
><br>
> Are we going to start gating on spellcheck of code and commit messages?  :-)<br>
<br>
</div>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.</blockquote><div><br></div><div>Spelling and grammar are two totally separate issues, and I think grammar is out of scope here. A non-human gate for basic spelling would be fantastic- faster feedback for the author and fewer wasted cycles by reviewers.<br>
</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">I'd much rather see us focus on things like... I<br>

dunno... maybe having the code actually work?<br></blockquote><div><br></div><div>Effectively communicating the intent and thinking behind the code is just as important as the code itself :)</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=""><div class="h5"><br>
><br>
> --<br>
> Russell Bryant<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><br clear="all"><div><br></div>-- <br><div><br></div>-Dolph
</div></div>