openstack/openstack-doc-tools
Code Issues Proposed changes

Remove setup.py check from pep8 job

Browse Source 
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 <sean.mcginnis@gmail.com>
This commit is contained in:
Sean McGinnis 2018-10-30 10:37:10 -05:00
parent a3fdc40659
commit 074a84df78
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
tox.ini
View File

@ -17,7 +17,6 @@ commands = stestr run --slowest {posargs}
[testenv:pep8] [testenv:pep8]
basepython = python3 basepython = python3
commands = commands =
python setup.py check --restructuredtext --strict
flake8 flake8
# Run doc8 to check .rst and .txt files. # Run doc8 to check .rst and .txt files.
# HACKING.rst is the only file that is not referenced from # HACKING.rst is the only file that is not referenced from