[openstack-dev] [Cinder] pep8 issues and how to pep8 locally ?

Deepak Shetty dpkshetty at gmail.com
Mon Apr 28 10:17:14 UTC 2014


Why is this not part of cinder or devstack dep and why isn't this auto
installed ?
I searched the HACKING and CONTRIBUTING docs.. none of them explain how to
sanity check your code before posting it to gerrit ... yum search and pip
install didn't help me install tox-epep8...
 How do i proceed ?


On Mon, Apr 28, 2014 at 3:39 PM, Sean Dague <sean at dague.net> wrote:

> On 04/28/2014 06:08 AM, Deepak Shetty wrote:
> > Hi,
> >
> > H703  Multiple positional placeholders
> >
> > I got this for one of my patch and googling i could find that the fix is
> > to use
> > dict instead of direct substitues.. which i did.. but it still gives me
> > the error :(
> >
> > Also just running pep8 locally on my glsuterfs.py file doesn't show any
> > issue
> > but gerrit does.
> > So how do i run the same pep8 that gerrit does locally on my box, so
> > that I don't end up resending new patches due to failed gerrit build
> > checks ?
>
> tox -epep8
>
>         -Sean
>
> --
> Sean Dague
> http://dague.net
>
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20140428/cf021d4a/attachment.html>


More information about the OpenStack-dev mailing list