[Openstack-stable-maint] PEP8

James E. Blair corvus at inaugust.com
Sun Dec 23 16:26:30 UTC 2012


Alan Pevec <apevec at gmail.com> writes:

> Dana 23. 12. 2012. 07:23 "Gary Kotton" <gkotton at redhat.com> je napisao:
>> A few days ago the pep8 version on the CI servers was updated. This
> causes patches in the Quantum stable branch to fail (not sure what the
> situation is with the other projects). Can you please take a look at
> https://review.openstack.org/18594.
>
> Not sure if that's a good idea for a stable branch.
> Instead of that, you cloud consider fixing pep8 version in test-requires,
> e.g. Keystone has pep8==1.3.3

Indeed.  The pep8 tool is not installed on Jenkins nodes, but rather
whatever version is specified in each project's tox.ini is installed in
the tox virtualenv for testing.  Like all project-level tests, you can
run the exact same setup on your workstation with:

  tox -e pep8

At the Grizzly summit, folks agreed that projects should choose a pep8
version early in the cycle and pin it.  To be effective, pep8 actually
needs to be pinned in tox.ini.  Here is a change that does it for
stable/folsom:

  https://review.openstack.org/#/c/18599/

You may want to make a similar change for master.

-Jim



More information about the Openstack-stable-maint mailing list