Skip to content

docs(skills): require test coverage verification in commit workflow#1431

Merged
piorpua merged 1 commit intomainfrom
docs/commit-skill-require-tests
Mar 19, 2026
Merged

docs(skills): require test coverage verification in commit workflow#1431
piorpua merged 1 commit intomainfrom
docs/commit-skill-require-tests

Conversation

@kaizhou-lab
Copy link
Copy Markdown
Collaborator

Summary

  • Add explicit test coverage verification as Step 2 in the commit skill, before running quality gates
  • If logic was added or changed, corresponding unit tests must be written first — not deferred to a follow-up commit
  • Updated Quick Reference to reflect the new step

Closes #1430

Add explicit test coverage check as Step 2 in the commit skill,
before running quality gates. When logic is added or changed,
corresponding unit tests must be written first — not deferred
to a follow-up commit. Quick Reference updated accordingly.
@piorpua piorpua merged commit 0bd95fc into main Mar 19, 2026
4 checks passed
@piorpua piorpua deleted the docs/commit-skill-require-tests branch March 19, 2026 06:29
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.

docs: commit skill should require test coverage verification before committing

2 participants