]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
pep8: cleaned up excludes
authorIhar Hrachyshka <ihrachys@redhat.com>
Wed, 21 Jan 2015 15:44:06 +0000 (16:44 +0100)
committerIhar Hrachyshka <ihrachys@redhat.com>
Fri, 23 Jan 2015 14:04:56 +0000 (15:04 +0100)
Don't over-exclude files from pep8 automation. Specifically, tools/*
should maintain common Python style as any other Neutron code.

Don't exclude every single dot-file/dot-dir separately but instead
apply .* wildcard.

Drop rally-scenarios exclusion. First, the directory is now rally-jobs,
so exclusion didn't work. Second, it's better to also apply pep8 checks
for those files (there are some Python files inside the directory).

Change-Id: Ife1d8a538c1bdddac30ca1c30c48a71564860624

tox.ini

diff --git a/tox.ini b/tox.ini
index b6bce897f81ffacd01e6826ec55898491e4eea5e..4154d6276530e52b0244b088b3a51a954f1c21d7 100644 (file)
--- a/tox.ini
+++ b/tox.ini
@@ -79,7 +79,7 @@ ignore = E125,E126,E128,E129,E265,H305,H404,H405
 show-source = true
 builtins = _
 # TODO(dougw) neutron/tests/unit/vmware exclusion is a temporary services split hack
-exclude = .venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build,tools,.ropeproject,rally-scenarios,neutron/tests/unit/vmware*
+exclude = \.*,build,dist,neutron/openstack/common/*,neutron/tests/unit/vmware*
 
 [testenv:pylint]
 deps =