Skip to content

Fail Codecov checks on low coverage - #100

Merged
namurphy merged 1 commit into
PlasmaPy:mainfrom
namurphy:codecov-targets
Nov 4, 2026
Merged

namurphy merged 1 commit into
PlasmaPy:mainfrom
namurphy:codecov-targets

Conversation

@namurphy

@namurphy namurphy commented Nov 4, 2026

Copy link
Copy Markdown
Member

Make the Codecov status checks fail instead of being informational only:

  • project: target: auto with threshold: 0.1%, so the check fails if a PR lowers overall coverage by more than 0.1% relative to the base commit.
  • patch: target: 95%, so the check fails if less than 95% of the lines changed in a PR are covered.

The updated .github/codecov.yml passes Codecov's validator (https://codecov.io/validate).

Notes:

  • Coverage for main doesn't currently seem to be recorded under PlasmaPy/pyfaradaycup on Codecov, and the uploads from main point to namurphy/pyfaradaycup. Until that's fixed, the project check may have no base to compare against. Regenerating CODECOV_TOKEN from the PlasmaPy repo's Codecov settings should fix it.
  • These checks won't block merging until they are required in a branch ruleset.

🤖 Generated with Claude Code

Make the project status fail when a change lowers overall coverage by
more than 0.1%, and the patch status fail when less than 95% of changed
lines are covered. Both statuses were previously informational only.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@namurphy
namurphy merged commit 3aed874 into PlasmaPy:main Nov 4, 2026
16 checks passed
@namurphy
namurphy deleted the codecov-targets branch November 4, 2026 19:38
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