Skip to content

ci: gate PRs on 100% patch coverage#134

Merged
mastermanas805 merged 1 commit into
mainfrom
ci/patch-coverage-100-gate
May 22, 2026
Merged

ci: gate PRs on 100% patch coverage#134
mastermanas805 merged 1 commit into
mainfrom
ci/patch-coverage-100-gate

Conversation

@mastermanas805
Copy link
Copy Markdown
Member

Summary

  • Adds diff-cover patch-coverage enforcement: every changed line in a PR diff must be covered by a test (--fail-under=100 vs origin/main).
  • vitest's v8 coverage provider emits lcov with src/*.ts(x) paths + source-mapped line numbers, lining up directly with the PR diff.
  • Adds @vitest/coverage-v8 devDependency (+ lockfile) so vitest run --coverage runs non-interactively in CI.
  • npm run gate left intact; fetch-depth: 0 so diff-cover resolves the base ref.

New permanent org mandate.

🤖 Generated with Claude Code

Adds diff-cover patch-coverage enforcement: every changed line in a PR must be
covered by a test (--fail-under=100 vs origin/main). vitest's v8 coverage
provider emits lcov with src/*.ts(x) paths + source-mapped line numbers, so it
lines up directly with the PR diff. @vitest/coverage-v8 is already a
devDependency on main (#132), so this PR only adds the workflow gate.
`npm run gate` untouched. New org mandate.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@mastermanas805 mastermanas805 force-pushed the ci/patch-coverage-100-gate branch from 318215e to 7fb4235 Compare May 22, 2026 02:32
@github-actions
Copy link
Copy Markdown

size-limit report 📦

Path Size
dist/assets/index-BwyF-Dau.js 143.87 KB (0%)
dist/assets/index-C9T0ot61.css 6.19 KB (0%)

@mastermanas805 mastermanas805 merged commit 8a2699b into main May 22, 2026
13 checks passed
@mastermanas805 mastermanas805 deleted the ci/patch-coverage-100-gate branch May 22, 2026 02:35
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.

2 participants