Skip to content

ci(codecov): gate the patch status with an explicit 95% target - #24

Merged
tyrsson merged 1 commit into
1.0.xfrom
tyrsson-codecov-explicit-patch-target
Nov 2, 2026
Merged

tyrsson merged 1 commit into
1.0.xfrom
tyrsson-codecov-explicit-patch-target

Conversation

@tyrsson

@tyrsson tyrsson commented Nov 1, 2026

Copy link
Copy Markdown
Member

Why

coverage.status.patch.default.target was auto. Codecov resolves auto from the base commit's coverage report. The org CI workflow at webinertia/.github, .github/workflows/continuous-integration.yml, runs on pull_request only, so a default branch can have no coverage report at all.

That leaves the patch gate in one of two bad states:

  • With no base report, Codecov posts a passing status titled "No report found to compare against", which gates nothing.
  • With a stale base report, the gate compares the diff against an arbitrary old snapshot.

Either way the gate is not measuring the diff.

Change

target: auto becomes target: 95%, with a comment recording why. 95 matches the fleet mutation gate (min_msi and min_covered_msi are both 95 in webware-ci.json).

threshold: 0% and base: auto are unchanged. coverage.status.project.default stays auto: a single shared number cannot serve a fleet whose project coverage varies widely.

Precedent

Already proven in webinertia/webware-tools#70 (merged as 4c3dbf0) and webinertia/asciidoc-php#19 (merged as 32bd9f2). After that fix, asciidoc-php-multi-repo#8 head 7ea871f reports codecov/patch as "97.44% of diff hit (target 95.00%)" instead of "No report found to compare against".

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@codecov

codecov Bot commented Nov 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@tyrsson
tyrsson merged commit 5234277 into 1.0.x Nov 2, 2026
12 checks passed
@tyrsson
tyrsson deleted the tyrsson-codecov-explicit-patch-target branch November 2, 2026 00:05
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