Skip to content

Commit

Permalink
chore: remove envdir settings from tox.ini
Browse files Browse the repository at this point in the history
In previous versions of `tox` it made sense to use `envdir` to have
multiple test environments share the same directory when they had the
same requirements.

But since `tox>=4` it no longer makes sense to do this as now `tox`
will detect a difference and keep recreating the environment and cause
it to be slower than having separate environments. What is happening
now is if you run `tox -e black,flake8` it will create the
environments. Then run `tox -e black,flake8` again it will again
recreate the environments. By removing `envdir` it won't recreate the
environments on the second and subsequent runs.
  • Loading branch information
JohnVillalovos committed Aug 17, 2023
1 parent 635fbad commit 2768fc3
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -9,13 +9,11 @@ deps = -r{toxinidir}/requirements-dev.txt

[testenv:black]
basepython = python3
envdir={toxworkdir}/lint
commands =
black {posargs} .

[testenv:flake8]
basepython = python3
envdir={toxworkdir}/lint
commands =
flake8 {posargs} .

Expand Down

0 comments on commit 2768fc3

Please sign in to comment.