[openstack-dev] How to add a periodic check for typos?

Amrith Kumar amrith at tesora.com
Fri Nov 20 13:29:02 UTC 2015


So, just for grins, I took this approach out for a spin on Trove and noticed this as part of the change proposed by topy.

-               "hebrew": ["hebrew_general_ci", "hebrew_bin"],
+               "Hebrew": ["hebrew_general_ci", "hebrew_bin"],

-               "greek": ["greek_general_ci", "greek_bin"],
+               "Greek": ["greek_general_ci", "greek_bin"],

In this particular case the change is being proposed in something that is a set of collation sequences and while "Hebrew" is the correct capitalization in the English language, what we need is "hebrew" in this case. Similarly for Greek and greek. 

If there were some way to specify a set of "required typos" I would be OK running this manually before I checked in code. I'm not sure I'd like it either as a tox or a hacking rule though because the pain may outweigh the gain.

-amrith


> -----Original Message-----
> From: Gareth [mailto:academicgareth at gmail.com]
> Sent: Thursday, November 19, 2015 10:13 PM
> To: OpenStack Development Mailing List (not for usage questions)
> <openstack-dev at lists.openstack.org>
> Subject: Re: [openstack-dev] How to add a periodic check for typos?
> 
> Just talking about the idea of auto-spelling-fix.
> 
> My example patch https://review.openstack.org/#/c/247261/ doesn't work.
> Topy fixes something and break others. So it is just okay to do auto-spelling-
> check now, not fix :(
> 
> 
> 
> On Fri, Nov 20, 2015 at 6:31 AM, Matt Riedemann
> <mriedem at linux.vnet.ibm.com> wrote:
> >
> >
> > On 11/18/2015 8:00 PM, Gareth wrote:
> >>
> >> Hi stacker,
> >>
> >> We could use some 3rd tools like topy:
> >>      pip install topy
> >>      topy -a <your project>
> >>      git commit & git review
> >>
> >> Here is an example: https://review.openstack.org/#/c/247261/
> >>
> >> Could we have a periodic job like Jenkins users updating our
> >> requirement.txt?
> >>
> >
> > Are you asking for all projects or just a specific project you work on
> > and you forgot to tag the subject line?
> >
> > I wouldn't have a bot doing this, if I were going to do it (which I
> > wouldn't for nova). You could have it built into your pep8 job, or a
> > separate job that is voting on your project, if you really care about spelling.
> >
> > --
> >
> > Thanks,
> >
> > Matt Riedemann
> >
> >
> >
> >
> __________________________________________________________
> ____________
> > ____ OpenStack Development Mailing List (not for usage questions)
> > Unsubscribe:
> > OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> 
> 
> 
> --
> Gareth
> 
> Cloud Computing, OpenStack, Distributed Storage, Fitness, Basketball
> OpenStack contributor, kun_huang at freenode My promise: if you find any
> spelling or grammar mistakes in my email from Mar 1 2013, notify me and I'll
> donate $1 or ¥1 to an open organization you specify.
> 
> __________________________________________________________
> ________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-
> request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


More information about the OpenStack-dev mailing list