Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Commit

Permalink
Detox, part 2 of N (#12152)
Browse files Browse the repository at this point in the history
I've argued in #11537 that poetry and tox don't cooperate well at the
moment. (See also #12119.) Therefore I'm pruning away bits of tox to make the transition to poetry easier. This change removes the commands for coverage.

We don't use coverage in anger at the moment. It shouldn't be too hard to add coverage as a dev-dependency and reintroduce this if we really want it.
  • Loading branch information
David Robertson authored Mar 3, 2022
1 parent 7e91107 commit 9297d04
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 26 deletions.
1 change: 1 addition & 0 deletions changelog.d/12152.misc
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Prune unused jobs from `tox` config.
26 changes: 0 additions & 26 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -158,32 +158,6 @@ commands =
extras = lint
commands = isort -c --df {[base]lint_targets}

[testenv:combine]
skip_install = true
usedevelop = false
deps =
coverage
pip>=10
commands=
coverage combine
coverage report

[testenv:cov-erase]
skip_install = true
usedevelop = false
deps =
coverage
commands=
coverage erase

[testenv:cov-html]
skip_install = true
usedevelop = false
deps =
coverage
commands=
coverage html

[testenv:mypy]
deps =
{[base]deps}
Expand Down

0 comments on commit 9297d04

Please sign in to comment.