Skip to content

Add codecov.yml with a 1% coverage threshold#683

Open
wbarnha wants to merge 1 commit into
masterfrom
claude/codecov-config
Open

Add codecov.yml with a 1% coverage threshold#683
wbarnha wants to merge 1 commit into
masterfrom
claude/codecov-config

Conversation

@wbarnha

@wbarnha wbarnha commented Jul 17, 2026

Copy link
Copy Markdown
Member

Description

The repo has no Codecov config, so Codecov applies its default project status — target: auto, threshold: 0% — which fails the codecov/project check on any coverage decrease at all. In practice the drops are sub-0.1% jitter from async/threaded lines being counted slightly differently between runs (not real regressions), and they've been forcing admin-merges on otherwise-green PRs.

This adds a root codecov.yml giving project and patch a 1% threshold — enough slack to absorb the noise while still flagging a genuine coverage regression — and disables the per-PR Codecov comment (the status checks already carry the signal).

coverage:
  status:
    project:
      default:
        target: auto
        threshold: 1%
    patch:
      default:
        target: auto
        threshold: 1%
comment: false

Notes:

  • Repo-wide policy change: it applies to all PRs once merged (it does not retroactively re-flip an already-run check; a re-run picks it up).
  • Validated as well-formed YAML with the standard Codecov schema.

Generated by Claude Code

The repo had no Codecov config, so Codecov applied its default project
status (target: auto, threshold: 0%) and failed CI on any coverage
decrease -- including the sub-0.1% jitter from async/threaded lines being
counted differently between runs, which forced admin-merges on otherwise
green PRs. Add a codecov.yml giving project and patch a 1% threshold so
the gate tolerates that noise while still catching real regressions, and
disable the per-PR comment.
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