Skip to content

enforce coverage 80#87

Merged
devlux76 merged 9 commits intomainfrom
enforce-coverage-80
Mar 14, 2026
Merged

enforce coverage 80#87
devlux76 merged 9 commits intomainfrom
enforce-coverage-80

Conversation

@devlux76
Copy link
Copy Markdown
Owner

  • feat: add coverage enforcement for tests and update CI configuration
  • Enforce minimum 80% test coverage in CI + local pre-push hook
  • Relax branch coverage enforcement while keeping 80% minimum line/function/statement coverage

Copilot AI review requested due to automatic review settings March 14, 2026 10:13
@github-actions github-actions bot added the layer: ci CI/CD pipeline and build tooling label Mar 14, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds test coverage enforcement to the Vitest setup so CI (and local pre-push) fail if overall coverage drops below an 80% baseline for lines/functions/statements.

Changes:

  • Introduces vitest.config.ts with V8 coverage reporting and 80% thresholds (branch threshold relaxed).
  • Adds test:coverage script plus Husky pre-push hook to run coverage locally.
  • Updates CI workflow to run coverage-enforced tests instead of plain unit tests.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
vitest.config.ts New Vitest configuration enabling coverage collection and setting thresholds/exclusions.
package.json Adds test:coverage, installs @vitest/coverage-v8, and sets up Husky via prepare.
bun.lock Locks newly added dependencies for coverage + Husky.
.husky/pre-push Runs bun run test:coverage before allowing pushes.
.github/workflows/ci.yml Switches CI test step to bun run test:coverage for enforcement.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 14, 2026

@devlux76 I've opened a new pull request, #88, to work on those changes. Once the pull request is ready, I'll request review from you.

devlux76 and others added 4 commits March 14, 2026 04:22
* Initial plan

* Update @vitest/coverage-v8 to latest to match all other dependencies

Co-authored-by: devlux76 <86517969+devlux76@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: devlux76 <86517969+devlux76@users.noreply.github.com>
@devlux76 devlux76 merged commit 6b0a5af into main Mar 14, 2026
3 checks passed
@devlux76 devlux76 deleted the enforce-coverage-80 branch March 14, 2026 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

layer: ci CI/CD pipeline and build tooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants