Require one concern per commit, and a bump of its own - #21
Merged
Merged
Conversation
The subject is printed beside every file the commit touched, so folding four unrelated changes into one prints a sentence that is a quarter true of each. Releasing 0.0.2-beta.3 put the same line on sixteen paths, CODE_OF_CONDUCT.md and .github/ among them, where it described nothing that had changed. The rule that was missing is about granularity, not wording — the wording rules above were followed.
The release commit's subject lands on every file it touched. 0.0.2-beta.3's also carried a Code of Conduct contact and a CI job, so that sentence became the description of CODE_OF_CONDUCT.md and .github/ — neither of which had anything to do with npm or the site. Checked against that commit, this rejects exactly those two paths.
The comment carried a status line — which integrations were out, which were not — beside the rule it was explaining. The rule is what a reader needs, and the status is asserted by tests/contract.test.mjs rather than by a sentence that goes stale between releases.
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
jsray | 622f6b1 | Commit Preview URL Branch Preview URL |
Sep 05 2026, 05:31 PM |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The rule that was missing when 0.0.2-beta.3's subject landed on sixteen paths, plus the check that enforces it in
tools/release.sh— verified against that commit, it rejects exactlyCODE_OF_CONDUCT.mdand.github/workflows/ci.yml.Also drops the status line from the footer comment; the status is asserted by
tests/contract.test.mjs.