Skip to content

Commit 0f70ecc

Browse files
skearnesclaude
andauthored
Remove Codecov from CI (#189)
Mirrors open-reaction-database/ord-schema#842: drops the external Codecov dependency. Codecov was purely informational here (no merge gating, no README badge) — it only provided the PR coverage comment and the codecov.io dashboard. - Drop the `codecov/codecov-action@v5` upload step in tests.yml, along with its fork/Dependabot token-auth workarounds and CODECOV_TOKEN / slug usage. - Replace `coverage xml` with `coverage report` so the coverage summary still prints in the job log — no external service required. There is no codecov.yml in this repo to delete. Coverage is still measured by `pytest --cov`; this just stops uploading it externally. Follow-up (outside this repo): remove the CODECOV_TOKEN org/repo secret and disable the Codecov GitHub App so it stops posting empty status checks. Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent 728975c commit 0f70ecc

1 file changed

Lines changed: 1 addition & 11 deletions

File tree

.github/workflows/tests.yml

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -53,17 +53,7 @@ jobs:
5353
run: |
5454
uv run coverage erase
5555
uv run pytest -vv --cov=ord_interface --durations=20 --ignore=ord_interface/editor
56-
uv run coverage xml
57-
# Codecov uses CODECOV_TOKEN when set. Do not fail the job when uploads cannot authenticate:
58-
# fork PRs (tokenless), Dependabot PRs (no repo secrets + protected dependabot/* branches).
59-
# Pushes to main and normal same-repo PRs stay strict.
60-
- uses: codecov/codecov-action@v5
61-
with:
62-
files: coverage.xml
63-
# CODECOV_TOKEN is an org-wide token, so the repo slug must be explicit.
64-
slug: open-reaction-database/ord-interface
65-
fail_ci_if_error: ${{ github.event_name == 'push' || (github.event_name == 'pull_request' && !github.event.pull_request.head.repo.fork && !startsWith(github.head_ref, 'dependabot/')) }}
66-
token: ${{ secrets.CODECOV_TOKEN }}
56+
uv run coverage report
6757
6858
test_app:
6959
strategy:

0 commit comments

Comments
 (0)