From aa3f21c8991dd027c288e0010a6905b069df6ac7 Mon Sep 17 00:00:00 2001 From: Sean McGinnis Date: Tue, 30 Oct 2018 10:11: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: If922868b53aaff788bc9439c0ce1b05340208704 Signed-off-by: Sean McGinnis --- tox.ini | 2 -- 1 file changed, 2 deletions(-) diff --git a/tox.ini b/tox.ini index 4c351db..0937984 100644 --- a/tox.ini +++ b/tox.ini @@ -20,8 +20,6 @@ commands = bindep test [testenv:pep8] commands = - # Check the readme for python release - python setup.py check --strict -r # Run hacking/flake8 check for all python files bash -c "git ls-files | grep -v releasenotes | xargs grep --binary-files=without-match \ --files-with-match '^.!.*python$' \