Skip to content

chore: add coverage enforcement#17

Merged
francoischalifour merged 5 commits into
mainfrom
fc/fix-knip-warnings
Jul 1, 2026
Merged

chore: add coverage enforcement#17
francoischalifour merged 5 commits into
mainfrom
fc/fix-knip-warnings

Conversation

@francoischalifour

Copy link
Copy Markdown
Member

Summary

  • Add Bun coverage configuration for the CLI package with an 80% line/function threshold.
  • Replace the plain unit-test step in CI and local verification with coverage-enabled test runs.
  • Upload lcov.info from GitHub Actions so coverage can be inspected after CI.
  • Tighten knip entrypoints by removing the CLI binary entry and the unused security binary allowance.
  • Update contributor guidance to reflect the new coverage check and HTML coverage command.

Testing

  • Updated the verification path to run coverage-enabled unit tests as part of the standard CLI gate.
  • CI now produces an lcov artifact for coverage inspection.
  • Documentation changes only beyond the test/verification workflow updates.

Risks

  • Coverage thresholds may fail CI if existing tests do not exercise enough code paths.
  • knip now relies on the test and script entrypoints only, so any future CLI entrypoint changes may require updating its config.

@francoischalifour francoischalifour merged commit 3bbb6a4 into main Jul 1, 2026
4 checks passed
@francoischalifour francoischalifour deleted the fc/fix-knip-warnings branch July 1, 2026 09:15
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