summaryrefslogtreecommitdiff
path: root/test-requirements.txt
diff options
context:
space:
mode:
authorClay Gerrard <clay.gerrard@gmail.com>2014-02-27 20:27:29 -0800
committerClay Gerrard <clay.gerrard@gmail.com>2014-03-04 08:29:38 -0800
commit3f857898a93081a1c17dd89850d0ece04569d120 (patch)
tree777c3f77fa6266bb4f4357fad1f6deafff4ad155 /test-requirements.txt
parent076634e23c9c9acbc48f71d1f28d32982f6d95a9 (diff)
downloadswift-3f857898a93081a1c17dd89850d0ece04569d120.tar.gz
Update tox.ini so flake8 works even without hacking
The flake8 command (when run from the command line or via vim-flake8) seems to bypass the the normal pep8 [W]arning and [E]rror as well as py[F]lake checks if the project level tox.ini includes a select, but doesn't include them in it's select option (tested on flake8 v2.0 and v2.1). Somehow if hacking is installed these checks always run even if they're not explicitly listed in the select list - but adding them so flake8 works on our project even without hacking doesn't seem to cause any problems. Also The [flake8] section's "select" option in tox.ini doesn't seem to run all checks when presented with the space-comma seperated list. I had to pull out the bare except check (H201), because our hacking fix for the "H201 # noqa" hasn't been released to pypi yet (maybe hacking version 0.9.0?). As far as I know the Swift core development team is fully onboard with pep8, and the flake8 checks are mostly like "unused import" stuff - so I *think* this change is more about making the tool test what we want it too and not changing or adding any new requirements or expectations. Change-Id: Ia911c63745100cce182d5c5eb67b77e31c1e9b18
Diffstat (limited to 'test-requirements.txt')
-rw-r--r--test-requirements.txt2
1 files changed, 1 insertions, 1 deletions
diff --git a/test-requirements.txt b/test-requirements.txt
index 7036e8d19..0e3414b8c 100644
--- a/test-requirements.txt
+++ b/test-requirements.txt
@@ -1,5 +1,5 @@
# Hacking already pins down pep8, pyflakes and flake8
-hacking>=0.5.6,<0.6
+hacking>=0.8.0,<0.9
coverage
nose
nosexcover