Skip to content

Commit

Permalink
ci: use "--cov-fail-under 100" rather than codecov
Browse files Browse the repository at this point in the history
Since this project has 100% statement coverage, we can use the simple
pytest-cov command-line option to ensure the project retains full
coverage on all incoming changes. There is no need to involve a remote
service for this.
  • Loading branch information
rohanpm committed Jan 21, 2025
1 parent a1f49b3 commit 376b08f
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 8 deletions.
7 changes: 0 additions & 7 deletions .github/workflows/tox-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,13 +45,6 @@ jobs:
run: pip install tox
- name: Run Tox
run: tox -e cov
- name: Install pytest cov
run: pip install pytest-cov
- name: Upload coverage to Codecov
uses: codecov/codecov-action@v2
with:
fail_ci_if_error: true
verbose: true
docs:
runs-on: ubuntu-latest
steps:
Expand Down
2 changes: 1 addition & 1 deletion tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ deps=
pytest-cov
usedevelop=true
commands=
pytest --cov-report=html --cov-report=xml --cov=src {posargs}
pytest --cov-report=html --cov-report=xml --cov=src --cov-fail-under 100 {posargs}

[testenv:docs]
deps=
Expand Down

0 comments on commit 376b08f

Please sign in to comment.