Browse Source

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
Andrew Laski 6 years ago
parent
commit
d7f1a71311
2 changed files with 3 additions and 2 deletions
  1. 2
    1
      run_tests.sh
  2. 1
    1
      tox.ini

+ 2
- 1
run_tests.sh View File

@@ -103,7 +103,8 @@ function copy_subunit_log {
103 103
 
104 104
 function run_pep8 {
105 105
   echo "Running pep8 ..."
106
-  srcfiles="--exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg ."
106
+  srcfiles="--exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*"
107
+  srcfiles+=",*egg,build ."
107 108
   # Just run PEP8 in current environment
108 109
   #
109 110
   ignore="--ignore=E12,E711,E721,E712"

+ 1
- 1
tox.ini View File

@@ -16,7 +16,7 @@ commands = python setup.py testr --testr-args='{posargs}'
16 16
 deps = pep8==1.3.3
17 17
 commands =
18 18
   pep8 --ignore=E12,E711,E721,E712 --show-source \
19
-    --exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg .
19
+    --exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build .
20 20
 
21 21
 [testenv:venv]
22 22
 commands = {posargs}

Loading…
Cancel
Save