From d7f1a71311dc8da730d281ac8c73618d3c00dc02 Mon Sep 17 00:00:00 2001 From: Andrew Laski Date: Mon, 25 Mar 2013 16:41:46 -0400 Subject: [PATCH] Don't check build/ for pep8 violations There's no need to run pep8 on the build dir, and it contains E502 false positives due to some part of the build process adding line continuations in places that they're not needed. Change-Id: I7ea19aea2b9e46503aa8acc06ce6b9d7ea18113a --- run_tests.sh | 3 ++- tox.ini | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/run_tests.sh b/run_tests.sh index a3a74cdcf..a42d0d3d6 100755 --- a/run_tests.sh +++ b/run_tests.sh @@ -103,7 +103,8 @@ function copy_subunit_log { function run_pep8 { echo "Running pep8 ..." - srcfiles="--exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg ." + srcfiles="--exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*" + srcfiles+=",*egg,build ." # Just run PEP8 in current environment # ignore="--ignore=E12,E711,E721,E712" diff --git a/tox.ini b/tox.ini index 4d5dbdcb1..e4458844a 100644 --- a/tox.ini +++ b/tox.ini @@ -16,7 +16,7 @@ commands = python setup.py testr --testr-args='{posargs}' deps = pep8==1.3.3 commands = pep8 --ignore=E12,E711,E721,E712 --show-source \ - --exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg . + --exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build . [testenv:venv] commands = {posargs}