summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMonty Taylor <mordred@inaugust.com>2013-05-18 09:40:15 -0700
committerMonty Taylor <mordred@inaugust.com>2013-05-18 09:40:15 -0700
commitb324ccc37bc6c0dfded22ba5b4eccc96642791ba (patch)
tree246cad4d497bef7819e9701867d5156f23239f67
parentf175a98e9fda8a68b592093040c2f7b8bd584d9b (diff)
downloadpython-ceilometerclient-b324ccc37bc6c0dfded22ba5b4eccc96642791ba.tar.gz
Migrate to flake8.
Fixes bug 1172444. Change-Id: Iba46d40a0e3ae0385ce51181f47295a2b520110d
-rwxr-xr-xrun_tests.sh9
-rw-r--r--tools/test-requires8
-rw-r--r--tox.ini10
3 files changed, 14 insertions, 13 deletions
diff --git a/run_tests.sh b/run_tests.sh
index aea2461..01ed56b 100755
--- a/run_tests.sh
+++ b/run_tests.sh
@@ -102,13 +102,8 @@ function copy_subunit_log {
}
function run_pep8 {
- echo "Running pep8 ..."
- 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"
- ${wrapper} pep8 ${ignore} --show-source ${srcfiles}
+ echo "Running flake8 ..."
+ ${wrapper} flake8
}
TESTRTESTS="testr run --parallel $testropts"
diff --git a/tools/test-requires b/tools/test-requires
index 06f5bd9..c4ca04d 100644
--- a/tools/test-requires
+++ b/tools/test-requires
@@ -1,10 +1,14 @@
-# This file is managed by openstack-depends
+# Install bounded pep8/pyflakes first, then let flake8 install
+pep8==1.4.5
+pyflakes==0.7.2
+flake8==2.0
+hacking>=0.5.3,<0.6
+
coverage>=3.6
discover
distribute>=0.6.24
fixtures>=0.3.12
mox>=0.5.3
-pep8==1.4.5
python-subunit
sphinx>=1.1.2
testrepository>=0.0.13
diff --git a/tox.ini b/tox.ini
index 27c8600..67b21ed 100644
--- a/tox.ini
+++ b/tox.ini
@@ -17,13 +17,15 @@ sitepackages = True
downloadcache = ~/cache/pip
[testenv:pep8]
-deps = pep8==1.4.5
-commands =
- pep8 --ignore=E12,E711,E721,E712 --show-source \
- --exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build .
+commands = flake8
[testenv:cover]
commands = python setup.py testr --coverage --testr-args='{posargs}'
[testenv:venv]
commands = {posargs}
+
+[flake8]
+ignore = E12,E711,E721,E712,F,H
+show-source = True
+exclude = .venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build