From a78ce2f932ffbfce4e1c045babf92a680da0f0ec Mon Sep 17 00:00:00 2001 From: Nguyen Van Trung Date: Wed, 23 May 2018 23:37:28 +0700 Subject: [PATCH] Allow to check pep8 along with D000 This commit going to remove --ignore D000 which had raised following issues: "D000 Cannot analyze code. Pygments package not found." And they also fixed in https://review.openstack.org/#/c/568729/ Change-Id: Ie23b07a2909482a4b36cfe9159f42f57747927cd --- test-requirements.txt | 1 + tox.ini | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/test-requirements.txt b/test-requirements.txt index 196337f75..152b46dc4 100644 --- a/test-requirements.txt +++ b/test-requirements.txt @@ -18,3 +18,4 @@ testresources>=2.0.0 # Apache-2.0/BSD testscenarios>=0.4 # Apache-2.0/BSD testtools>=2.2.0 # MIT stestr>=1.0.0 # Apache-2.0 +Pygments>=2.2.0 # BSD license diff --git a/tox.ini b/tox.ini index 7fd0cf45f..2ff1f7d8e 100644 --- a/tox.ini +++ b/tox.ini @@ -25,7 +25,7 @@ passenv = http_proxy HTTP_PROXY https_proxy HTTPS_PROXY no_proxy NO_PROXY [testenv:pep8] commands = - doc8 -e .rst doc/source/ CONTRIBUTING.rst HACKING.rst README.rst --ignore D000 + doc8 -e .rst doc/source/ CONTRIBUTING.rst HACKING.rst README.rst bash tools/flake8wrap.sh {posargs} # The following bandit tests are being skipped: # B303 - Use of insecure MD2, MD4, or MD5 hash function.