Skip to content

Commit

Permalink
Correct the tox option for skipping sdist generation
Browse files Browse the repository at this point in the history
The tox option to skip source distribution building is skipsdist,
but this seems to be often misspelled skipdist instead, which gets
silently ignored and so does not take effect. Correct it
everywhere, in hopes that new projects will finally stop copying
this mistake around.

See https://tox.readthedocs.io/en/latest/config.html#conf-skipsdist
and tox-dev/tox#1388 for details.

Change-Id: Ieee8041dd177b28d74a0f175fb6df340d158c3c9
  • Loading branch information
fungi authored and Federico Ressi committed Jun 30, 2021
1 parent d2bbd93 commit 1f7ad83
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -236,7 +236,7 @@ setenv =
# package available only for /usr/bin/python interpreter
basepython = {env:IR_PYTHON:python3}
usedevelop = false
skipdist = true
skipsdist = true
skip_install = true
sitepackages = true
whitelist_externals =
Expand Down Expand Up @@ -294,7 +294,7 @@ commands =
# dependencies are missing! This also means that bindep must be installed
# separately, outside of the requirements files.
usedevelop = false
skipdist = true
skipsdist = true
skip_install = true


Expand All @@ -313,7 +313,7 @@ commands =
# dependencies are missing! This also means that bindep must be installed
# separately, outside of the requirements files.
usedevelop = false
skipdist = true
skipsdist = true
skip_install = false


Expand All @@ -328,7 +328,7 @@ commands =
# dependencies are missing! This also means that bindep must be installed
# separately, outside of the requirements files.
usedevelop = false
skipdist = true
skipsdist = true
skip_install = true


Expand All @@ -342,7 +342,7 @@ basepython = {[testenv:py3]basepython}
# dependencies are missing! This also means that bindep must be installed
# separately, outside of the requirements files.
usedevelop = false
skipdist = true
skipsdist = true
skip_install = true
deps = bindep
commands = bindep {posargs:test}
Expand Down

0 comments on commit 1f7ad83

Please sign in to comment.