Switch to Hacking 0.8 / Pep8 1.4.x
Most other projects use Hacking and flake8 for the Pep8 checks, which provides signficant advantages: * Many more checks for typical violations of OpenStack coding guidelines (less manual work for reviewers, less style nitpicking) * more bug fixes in the pep8 checks * Pyflakes checks - which find usual real coding bugs Currently most of the checks are disabled via the ignore section in tox.ini, to be reenabled in later commits. Change-Id: Iec4d20caf640bd5ca9d796702a6bb1085c60933e
This commit is contained in:
parent
78f620d80b
commit
7750a08e91
|
@ -1,9 +1,9 @@
|
|||
hacking>=0.8.0,<0.9
|
||||
coverage>=3.6
|
||||
discover
|
||||
fixtures>=0.3.14
|
||||
mock>=1.0
|
||||
ordereddict
|
||||
pep8==1.3.3
|
||||
sphinx>=1.2.1,<1.3
|
||||
testrepository>=0.0.18
|
||||
testtools>=0.9.34
|
||||
|
|
9
tox.ini
9
tox.ini
|
@ -12,8 +12,8 @@ deps = -r{toxinidir}/requirements.txt
|
|||
commands = python setup.py testr --testr-args='{posargs}'
|
||||
|
||||
[testenv:pep8]
|
||||
deps = pep8
|
||||
commands = pep8 --repeat --show-source manilaclient setup.py
|
||||
commands =
|
||||
flake8
|
||||
|
||||
[testenv:venv]
|
||||
commands = {posargs}
|
||||
|
@ -23,3 +23,8 @@ commands = python setup.py testr --coverage --testr-args='{posargs}'
|
|||
|
||||
[tox:jenkins]
|
||||
downloadcache = ~/cache/pip
|
||||
|
||||
[flake8]
|
||||
ignore = E12,H102,H23,H30,H40,H501,F
|
||||
builtins = _
|
||||
exclude = .venv,.tox,dist,doc,openstack,*egg
|
||||
|
|
Loading…
Reference in New Issue