From 50a1ed71cb73384379f7cef173e3e7f4158c21fb Mon Sep 17 00:00:00 2001 From: Sean McGinnis Date: Tue, 30 Oct 2018 10:11:48 -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: Ia981b27852b9061c8a406e68388c60595aa528f8 Signed-off-by: Sean McGinnis --- tox.ini | 1 - 1 file changed, 1 deletion(-) diff --git a/tox.ini b/tox.ini index 914a297875d..b9cf6c77298 100644 --- a/tox.ini +++ b/tox.ini @@ -70,7 +70,6 @@ setenv = [testenv:pep8] basepython = python3 commands = - python setup.py check --restructuredtext --strict flake8 {posargs} . {toxinidir}/tools/config/check_uptodate.sh {toxinidir}/tools/check_exec.py {toxinidir}/cinder {toxinidir}/doc/source/ {toxinidir}/releasenotes/notes