Skip to content

Commit 42a934e

Browse files
DirectXMan12frozencemetery
authored andcommitted
Make tox use lib-verify
The tox config previously had a copy of the same set of skipped checks as the Travis verify functions. In order to avoid issues in the future, it now just calls into that.
1 parent 358a92f commit 42a934e

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

.travis/lib-verify.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ verify::flake8() {
1111
# E225: missing whitespace around operator
1212
# E226: missing whitespace around arithmetic operator
1313
# E227: missing whitespace around bitwise or shift operator
14-
# E402: module level import not at top of file
14+
# E402: module level import not at top of file (needed for the `GSSAPI="blah" lines)
1515
# E901: SyntaxError or IndentationError
1616
# E999: Internal AST compilation error (flake8 specific)
1717
flake8 gssapi --filename='*.pyx,*.pxd' --ignore=E225,E226,E227,E402,E901,E999

tox.ini

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,9 @@
77
envlist = py27,py33,py34,py35
88

99
[testenv]
10-
# NB(sross): disabling E225,E226,E227,E901 make pep8 think Cython is ok
11-
# NB(sross): disable E402 because of the GSSAPI="blah" lines needed by Cython
10+
whitelist_externals=bash
1211
commands =
13-
flake8 setup.py
14-
flake8 gssapi
15-
flake8 gssapi --filename='*.pyx,*.pxd' --ignore=E225,E226,E227,E901,E402
12+
bash -c "source ./.travis/lib-verify.sh && verify::flake8"
1613
python setup.py nosetests []
1714

1815
deps = -r{toxinidir}/test-requirements.txt

0 commit comments

Comments
 (0)