]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Move pylint checks to pep8 testenv
authorarmando-migliaccio <armamig@gmail.com>
Wed, 25 Feb 2015 17:07:18 +0000 (09:07 -0800)
committerarmando-migliaccio <armamig@gmail.com>
Thu, 26 Feb 2015 01:45:38 +0000 (17:45 -0800)
commitc0aa0b37c0911558a2d91f1abff34e32dfe59bbe
tree11581d1902ad0b7be8fa2b0329bb3f6773d294ac
parent55c6bf9cc5962e93b71908c79d5fd15be2aff540
Move pylint checks to pep8 testenv

This change proposes to move pylint violation checks to the pep8
testenv. This changes make pylint gating within Neutron as it would
participate in the vote.  Having pylint executed on a separate job makes
it difficult to handle potential unexpected breakages, because we need
to get infra involved. When we need to renable the job, it is equally
painful.

Furthermore, it also causes us to spin an extra node, when the checks
can easily be handled by the node for the pep8 job.

Finally, having pylint running with tox -epep8 "helps" developers
become aware of pylint violations sooner rather than later, if they
"forget" to run the pylint testenv too before submitting the change.

In order to make this patch succeed, a couple of pylint violation
checks were skipped, as they slipped in, whilst the job was non-voting.

We'll tackle them in due course.

Change-Id: Ida6ae44a837d1761f5ce3e8a92f8850407f5cd16
.pylintrc
tox.ini