openstack/metalsmith
Code Issues Proposed changes

Move pep8 requirements to tox.ini

Browse Source 
We don't need them for unit tests and they're not usually constrained.

Change-Id: Ib1f9235c339df02f2e8c27a46700c9fb1b4be826
This commit is contained in:
Dmitry Tantsur 2020-12-09 14:38:01 +01:00
parent 1c06881c32
commit 912e0a37e5
2 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
test-requirements.txt
View File

@ -2,11 +2,5 @@
# of appearance. Changing the order has an impact on the overall integration
# process, which may cause wedges in the gate later.
coverage!=4.4,>=4.0 # Apache-2.0
doc8>=0.6.0 # Apache-2.0
flake8-import-order>=0.17.1 # LGPLv3
hacking>=3.1.0,<4.0.0 # Apache-2.0
stestr>=1.0.0 # Apache-2.0
Pygments>=2.2.0 # BSD
ansible>=2.8
pycodestyle>=2.0.0,<2.7.0 # MIT

6
tox.ini
View File

@ -42,6 +42,12 @@ passenv = http_proxy HTTP_PROXY https_proxy HTTPS_PROXY no_proxy NO_PROXY \
OS_USER_DOMAIN_NAME OS_PROJECT_DOMAIN_NAME OS_CLOUD OS_CACERT
[testenv:pep8]
deps =
Pygments>=2.2.0 # BSD
doc8>=0.6.0 # Apache-2.0
flake8-import-order>=0.17.1 # LGPLv3
hacking>=3.1.0,<4.0.0 # Apache-2.0
pycodestyle>=2.0.0,<2.7.0 # MIT
commands =
flake8 metalsmith
doc8 README.rst doc/source metalsmith_ansible/roles