Skip to content

ci: drop codecov upload step#23

Merged
lesnik512 merged 1 commit into
mainfrom
ci/drop-codecov
Jun 9, 2026
Merged

ci: drop codecov upload step#23
lesnik512 merged 1 commit into
mainfrom
ci/drop-codecov

Conversation

@lesnik512

Copy link
Copy Markdown
Member

Summary

Removes the codecov/codecov-action@v4.0.1 upload step from main.yml. Coverage floor is already enforced by --cov-fail-under=100 in pyproject.toml's [tool.pytest.ini_options], so codecov-action wasn't guarding anything that the test run doesn't already catch.

Why now

The recent CI bump-pins PR (#22) surfaced a GitHub deprecation warning: codecov-action@v4.0.1 runs on Node.js 20, which GitHub force-upgrades to Node.js 24 on 2026-06-16. Rather than carry that risk for a step that adds no enforcement, drop it.

Operator follow-up

  • The CODECOV_TOKEN repo secret is now unused. Can be deleted from Settings → Secrets and variables → Actions whenever convenient.

Test plan

  • PR CI green; pytest still fails the build below 100% coverage via the existing --cov-fail-under=100 in pyproject.toml.

🤖 Generated with Claude Code

Coverage floor is already enforced by --cov-fail-under=100 in
pyproject.toml's [tool.pytest.ini_options], so codecov-action wasn't
guarding anything. Drops the step ahead of GitHub's 2026-06-16 Node 20
deprecation that would have force-upgraded codecov-action@v4.0.1 to
Node 24 with unclear effects.

Operator follow-up: the CODECOV_TOKEN repo secret is now unused and
can be deleted from Settings -> Secrets and variables -> Actions.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@lesnik512 lesnik512 merged commit c341ea0 into main Jun 9, 2026
2 checks passed
@lesnik512 lesnik512 deleted the ci/drop-codecov branch June 9, 2026 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant