j4k-oss/conventional-commit-msg
Watch
2
0
Fork
You've already forked conventional-commit-msg
 0
Code Issues Pull requests Releases 3 Packages 1 Activity Actions

docs: make the CI example fail on an invalid commit #3

Merged
jercik merged 2 commits from docs/ci-example-exit-status into main 2026-10-22 17:55:15 +00:00
Conversation 1 Commits 2 Files changed 1 +2 -2
jercik commented 2026-10-22 17:33:34 +00:00
Owner

Follow-up to #2: the "for CI scripts" example exited 0 on an invalid last commit, because || echo "Invalid" succeeds; the invalid branch now exits 1.

This edits the same two README lines as #2 (the npx -y prefix), so merge #2 first and rebase this onto it.

🤖 Generated with Claude Code

Follow-up to #2: the "for CI scripts" example exited 0 on an invalid last commit, because `|| echo "Invalid"` succeeds; the invalid branch now exits 1. This edits the same two README lines as #2 (the `npx -y` prefix), so merge #2 first and rebase this onto it. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
docs: make the CI example fail on an invalid commit
All checks were successful
Checks / pr-flow-canary (pull_request) Has been skipped
Details
Checks / reuse (pull_request) Successful in 4s
Details
commit-msg / commitlint (pull_request) Successful in 10s
Details
Checks / quality-checks (26.5.0) (pull_request) Successful in 27s
Details
Checks / quality-checks (24.15.0) (pull_request) Successful in 31s
Details
Review / Review (pull_request_target) Successful in 2m1s
Details
16d81628da
 
The README's "for CI scripts" example ended in `|| echo "Invalid"`, so an
invalid last commit printed `Invalid` and the command list still exited 0. The
invalid branch now exits 1.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
forgejo-actions commented 2026-10-22 17:35:33 +00:00

Review 01M5JBTK1PRAWD7H2HHKJD711W — head e5193835176e63f008d4c45e68a70c961113f028

Review — j4k-oss/conventional-commit-msg @ 0886d66138

Scope: diff against base tree 6a911101974a
Status: dispatched — coverage complete (5/5 slots terminal)
Facts: current review-wide projection

Computed under:

{
  "abandonment": "abandonment-v1",
  "anchor_recipe": 1,
  "batch_policy": "batch-v1",
  "coverage": "coverage-v3",
  "dispatch_policy": "dispatch-v2",
  "grounder_version": 1,
  "grounding_read_rule": "grounding-read-v1",
  "promotion_policy": "promotion-v1",
  "report": "report-v4",
  "tally": "tally-v1",
  "triage_settle": "triage-settle-v2"
}

Findings (0)

No findings survived.

Reviewed:

  • restated-sets (whole/default): no-claims
  • general-bug (whole/default): no-claims
  • writing-quality (whole/default): no-claims
  • project-docs (whole/default): no-claims
  • unix-cli (whole/default): no-claims
  • test-trimming: skipped-by-dispatch — no tests changed; only README example edited
  • use-effect: skipped-by-dispatch — documentation-only change with no React Effects
  • react: skipped-by-dispatch — documentation-only change with no React components

Other claims

  • grounding-pending (0)
  • ungrounded (0)
  • rejected (0)
  • duplicate-of (0)
  • unadjudicated (0)

Coverage

Coverage pass: 01M5JBV0ZNZS9THZDG3SNV5GD9
Accounting: complete
Slot health: healthy

lens part arm unit status runs loss
restated-sets whole default no-claims 1 no
general-bug whole default no-claims 1 no
writing-quality whole default no-claims 1 no
project-docs whole default no-claims 1 no
unix-cli whole default no-claims 1 no
  • test-trimming — skipped-by-dispatch: no tests changed; only README example edited
  • use-effect — skipped-by-dispatch: documentation-only change with no React Effects
  • react — skipped-by-dispatch: documentation-only change with no React components
<!-- review:summary --> **Review** `01M5JBTK1PRAWD7H2HHKJD711W` — head `e5193835176e63f008d4c45e68a70c961113f028` # Review — j4k-oss/conventional-commit-msg @ 0886d6613886 Scope: diff against base tree `6a911101974a` Status: dispatched — coverage complete (5/5 slots terminal) Facts: current review-wide projection Computed under: ```json { "abandonment": "abandonment-v1", "anchor_recipe": 1, "batch_policy": "batch-v1", "coverage": "coverage-v3", "dispatch_policy": "dispatch-v2", "grounder_version": 1, "grounding_read_rule": "grounding-read-v1", "promotion_policy": "promotion-v1", "report": "report-v4", "tally": "tally-v1", "triage_settle": "triage-settle-v2" } ``` ## Findings (0) No findings survived. Reviewed: - restated-sets (whole/default): no-claims - general-bug (whole/default): no-claims - writing-quality (whole/default): no-claims - project-docs (whole/default): no-claims - unix-cli (whole/default): no-claims - test-trimming: skipped-by-dispatch — no tests changed; only README example edited - use-effect: skipped-by-dispatch — documentation-only change with no React Effects - react: skipped-by-dispatch — documentation-only change with no React components ## Other claims - grounding-pending (0) - ungrounded (0) - rejected (0) - duplicate-of (0) - unadjudicated (0) ## Coverage Coverage pass: 01M5JBV0ZNZS9THZDG3SNV5GD9 Accounting: complete Slot health: healthy | lens | part | arm | unit status | runs | loss | | --- | --- | --- | --- | --- | --- | | restated-sets | whole | default | no-claims | 1 | no | | general-bug | whole | default | no-claims | 1 | no | | writing-quality | whole | default | no-claims | 1 | no | | project-docs | whole | default | no-claims | 1 | no | | unix-cli | whole | default | no-claims | 1 | no | - test-trimming — skipped-by-dispatch: no tests changed; only README example edited - use-effect — skipped-by-dispatch: documentation-only change with no React Effects - react — skipped-by-dispatch: documentation-only change with no React components
🚀 1
Merge branch 'main' into docs/ci-example-exit-status
All checks were successful
Checks / pr-flow-canary (pull_request) Has been skipped
Details
Checks / reuse (pull_request) Successful in 5s
Details
commit-msg / commitlint (pull_request) Successful in 10s
Details
Checks / quality-checks (24.15.0) (pull_request) Successful in 30s
Details
Checks / quality-checks (26.5.0) (pull_request) Successful in 32s
Details
Review / Review (pull_request_target) Successful in 1m20s
Details
e519383517
 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
jercik merged commit db5341c840 into main 2026-10-22 17:55:15 +00:00
jercik deleted branch docs/ci-example-exit-status 2026-10-22 17:55:16 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
j4k-oss/conventional-commit-msg!3
No description provided.