From 074a84df78d9687c925d24cee074e6ceb772a495 Mon Sep 17 00:00:00 2001 From: Sean McGinnis Date: Tue, 30 Oct 2018 10:37:10 -0500 Subject: [PATCH] Remove setup.py check from pep8 job Using "python setup.py check -r -s" method of checking the package has been deprecated with the new recommendation to build the sdist and wheel, then running "twine check" against the output. Luckily, there is already a job that covers this that only runs when the README, setup.py, or setup.cfg files change, making running this in the pep8 job redundant. This covered by the test-release-openstack-python3 that is defined in the publish-to-pypi-python3 template. More details can be found in this mailing list post: http://lists.openstack.org/pipermail/openstack-dev/2018-October/136136.html Change-Id: I359d557dbf3438d219bed0303b7a95ad93192f46 Signed-off-by: Sean McGinnis --- tox.ini | 1 - 1 file changed, 1 deletion(-) diff --git a/tox.ini b/tox.ini index 57c8cb31..6d0e811f 100644 --- a/tox.ini +++ b/tox.ini @@ -17,7 +17,6 @@ commands = stestr run --slowest {posargs} [testenv:pep8] basepython = python3 commands = - python setup.py check --restructuredtext --strict flake8 # Run doc8 to check .rst and .txt files. # HACKING.rst is the only file that is not referenced from