zuul/nodepool
Code Issues Proposed changes

Stop using hacking for anything

Browse Source 
We've had it in here as a way to pin pycodestyle. But that's not
worth the energy to deal with things like hacking now spewing
incorrect messages about our apache license headers not being
right even though they are. (it's because sdague dedcided that
nobody has more than 11 lines of header)

zuul just uses a bare flake8. Match that choice.

Change-Id: I5bc847d151801fd31281389ee65184e964ddd2a5
Depends-On: https://review.opendev.org/712778
This commit is contained in:
Monty Taylor
2020-03-12 14:49:19 -05:00
parent 6c915cdbd3
commit 325455dfd6
2 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
test-requirements.txt
View File

@@ -1,5 +1,4 @@
# Nodepool uses hacking for its dependencies
hacking>=0.10.2,<0.11
flake8
coverage
fixtures>=0.3.12
mock>=1.0

2
tox.ini
View File

@@ -63,6 +63,6 @@ commands = stestr --test-path ./nodepool/tests/functional/openshift run --no-sub
[flake8]
# These are ignored intentionally in zuul projects;
# please don't submit patches that solely correct them or enable them.
ignore = E124,E125,E129,E402,E741,H,W503,W504
ignore = E124,E125,E129,E402,E741,W503,W504
show-source = True
exclude = .venv,.tox,dist,doc,build,*.egg