Skip to content

chore(ci): ratchet the coverage baseline to the measured 58.8% - #2319

Merged
rubenvdlinde merged 1 commit into
developmentfrom
chore/coverage-baseline-ratchet
Aug 4, 2026
Merged

chore(ci): ratchet the coverage baseline to the measured 58.8%#2319
rubenvdlinde merged 1 commit into
developmentfrom
chore/coverage-baseline-ratchet

Conversation

@rubenvdlinde

Copy link
Copy Markdown
Contributor

Coverage Baseline Check is red on development — because coverage improved past the recorded baseline: 55.54 recorded, 58.8 measured.

The gate is a ratchet and fails in both directions on purpose. That is right, but it means a rise nobody commits leaves development red — and the next genuine regression then lands on an already-red branch where nobody can distinguish it from the noise. A permanently-red gate is a gate that has stopped working.

Committing the recomputed value is precisely what the job's own error message asks for:

.coverage-baseline is out of date — measured coverage has improved past it. Commit the recomputed value […] The Coverage Baseline Protection job accepts any value that does not LOWER the baseline.

So this tightens the floor rather than relaxing it.

Found while clearing red CI across the fleet; unrelated to the change that produced the improvement.

🤖 Generated with Claude Code

`Coverage Baseline Check` has been red on development because coverage IMPROVED
past the recorded baseline: 55.54 recorded, 58.8 measured. The gate is a
ratchet — it fails in BOTH directions on purpose, so a rise that nobody commits
leaves development red and the next real regression lands on an already-red
branch where nobody can see it.

Committing the recomputed value is exactly what the job's own error message
asks for. The protection job accepts any value that does not LOWER the
baseline, so this tightens the floor rather than relaxing it.
@rubenvdlinde
rubenvdlinde requested a review from a team as a code owner August 4, 2026 01:41
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Quality Report — ConductionNL/openregister @ d84b95a

Check PHP Vue Security License Tests
lint
phpcs
phpmd
psalm
phpstan
phpmetrics
eslint
stylelint
build
composer ✅ 173/173
npm ✅ 713/713
PHPUnit
Newman
Playwright

Quality workflow — 2026-08-04 01:48 UTC

Download the full PDF report from the workflow artifacts.

@rubenvdlinde
rubenvdlinde merged commit a22d33b into development Aug 4, 2026
30 checks passed
@rubenvdlinde
rubenvdlinde deleted the chore/coverage-baseline-ratchet branch August 4, 2026 01:50
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