Skip to content

Commit

Permalink
fixes pytest-dev#8133: migrate to setuptools_scm 5.x and use SETUPTOO…
Browse files Browse the repository at this point in the history
…LS_SCM_PRETEND_VERSION_FOR_PYTEST
  • Loading branch information
RonnyPfannschmidt committed Dec 20, 2020
1 parent 89dcfbf commit 6c9a995
Show file tree
Hide file tree
Showing 6 changed files with 10 additions and 10 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/main.yml
Expand Up @@ -190,7 +190,7 @@ jobs:
- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install --upgrade wheel setuptools tox
pip install --upgrade wheel setuptools setuptools_scm[toml] tox
- name: Build package
run: |
python setup.py sdist bdist_wheel
Expand Down
1 change: 1 addition & 0 deletions changelog/8133.trivial.rst
@@ -0,0 +1 @@
Migrate to setuptools_scm 5.x to use SETUPTOOLS_SCM_PRETEND_VERSION_FOR_PYTEST for more robust release tooling.
2 changes: 1 addition & 1 deletion pyproject.toml
Expand Up @@ -2,7 +2,7 @@
requires = [
# sync with setup.py until we discard non-pep-517/518
"setuptools>=42.0",
"setuptools-scm[toml]>=3.4",
"setuptools-scm[toml]>=5.0.1",
"wheel",
]
build-backend = "setuptools.build_meta"
Expand Down
2 changes: 1 addition & 1 deletion scripts/release.py
Expand Up @@ -63,7 +63,7 @@ def regen(version):
print(f"{Fore.CYAN}[generate.regen] {Fore.RESET}Updating docs")
check_call(
["tox", "-e", "regen"],
env={**os.environ, "SETUPTOOLS_SCM_PRETEND_VERSION": version},
env={**os.environ, "SETUPTOOLS_SCM_PRETEND_VERSION_FOR_PYTEST": version},
)


Expand Down
2 changes: 1 addition & 1 deletion setup.cfg
Expand Up @@ -55,7 +55,7 @@ package_dir =
=src
setup_requires =
setuptools>=>=42.0
setuptools-scm>=3.4
setuptools-scm>=5.0.1
zip_safe = no

[options.entry_points]
Expand Down
11 changes: 5 additions & 6 deletions tox.ini
Expand Up @@ -16,13 +16,16 @@ envlist =
docs
docs-checklinks



[testenv]
commands =
{env:_PYTEST_TOX_COVERAGE_RUN:} pytest {posargs:{env:_PYTEST_TOX_DEFAULT_POSARGS:}}
doctesting: {env:_PYTEST_TOX_COVERAGE_RUN:} pytest --doctest-modules --pyargs _pytest
coverage: coverage combine
coverage: coverage report -m
passenv = USER USERNAME COVERAGE_* PYTEST_ADDOPTS TERM
passenv = USER USERNAME COVERAGE_* PYTEST_ADDOPTS TERM SETUPTOOLS_SCM_PRETEND_VERSION_FOR_PYTEST
setenv =
_PYTEST_TOX_DEFAULT_POSARGS={env:_PYTEST_TOX_POSARGS_DOCTESTING:} {env:_PYTEST_TOX_POSARGS_LSOF:} {env:_PYTEST_TOX_POSARGS_XDIST:}

Expand Down Expand Up @@ -83,10 +86,7 @@ commands =
[testenv:regen]
changedir = doc/en
basepython = python3
passenv = SETUPTOOLS_SCM_PRETEND_VERSION
# TODO: When setuptools-scm 5.0.0 is released, use SETUPTOOLS_SCM_PRETEND_VERSION_FOR_PYTEST
# and remove the next line.
install_command=python -m pip --use-deprecated=legacy-resolver install {opts} {packages}
passenv = SETUPTOOLS_SCM_PRETEND_VERSION_FOR_PYTEST
deps =
dataclasses
PyYAML
Expand All @@ -112,8 +112,6 @@ changedir = testing/plugins_integration
deps = -rtesting/plugins_integration/requirements.txt
setenv =
PYTHONPATH=.
# due to pytest-rerunfailures requiring 6.2+; can be removed after 6.2.0
SETUPTOOLS_SCM_PRETEND_VERSION=6.2.0a1
commands =
pip check
pytest bdd_wallet.py
Expand Down Expand Up @@ -187,6 +185,7 @@ extend-ignore =
; Docstring Content Issues
D400,D401,D401,D402,D405,D406,D407,D408,D409,D410,D411,D412,D413,D414,D415,D416,D417


[isort]
; This config mimics what reorder-python-imports does.
force_single_line = 1
Expand Down

0 comments on commit 6c9a995

Please sign in to comment.