← Back to team overview

openstack team mailing list archive

Re: Openstack common

 

There's a new version of pep8 out today (1.3.1) which fixes a few
indentation cases of if statements that were broken in 1.3.

Sergio

On Sun, Jun 17, 2012 at 9:01 PM, Adrian Smith <adrian@xxxxxxxx> wrote:

> pep8 1.3 (released 15-Jun) is much stricter about the indentation used
> on continuation lines.
>
> After upgrading we started seeing quite a few instances of E127,E128...
>
> "E127 continuation line over-indented for visual indent".
>
> Adrian
>
>
> On 17 June 2012 17:52, Jay Pipes <jaypipes@xxxxxxxxx> wrote:
> > What version of pep8 are you using? The errors look to be warnings that
> are
> > no longer printed in more modern versions of pep8...
> >
> > All the best,
> > -jay
> >
> >
> > On 06/17/2012 03:42 AM, Gary Kotton wrote:
> >>
> >> Hi,
> >> Over the weekend patches were made to Quantum to support Pep 1.3.
> >> Some of the patches were in the openstack common code. I have opened a
> >> bug to address this in the openstack common code
> >> (https://bugs.launchpad.net/openstack-common/+bug/1014216)
> >> I am currently updating the common code and will hopefully have a patch
> >> soon to address this for review.
> >> Thanks
> >> Gary
> >>
> >> _______________________________________________
> >> Mailing list: https://launchpad.net/~openstack
> >> Post to : openstack@xxxxxxxxxxxxxxxxxxx
> >> Unsubscribe : https://launchpad.net/~openstack
> >> More help : https://help.launchpad.net/ListHelp
> >
> >
> > _______________________________________________
> > Mailing list: https://launchpad.net/~openstack
> > Post to     : openstack@xxxxxxxxxxxxxxxxxxx
> > Unsubscribe : https://launchpad.net/~openstack
> > More help   : https://help.launchpad.net/ListHelp
>
> _______________________________________________
> Mailing list: https://launchpad.net/~openstack
> Post to     : openstack@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~openstack
> More help   : https://help.launchpad.net/ListHelp
>

References