chore(ci): dogfood the chore(release) skip filter#80
Merged
Conversation
monorel's own `ci/github/README.md` documents a recipe under
"Recipes -> Skipping CI on chore(release) commits" telling
downstream consumers to put
if: github.event_name == 'pull_request' || !startsWith(github.event.head_commit.message, 'chore(release):')
on every job that runs `go mod tidy` or otherwise resolves Go
module versions. monorel's own CI didn't follow that recipe.
Apply the filter to the three jobs in ci.yml that resolve module
versions (test, lint, tidy) and to the e2e job in e2e-gitea.yml.
The two PR-only jobs in ci.yml (commit-lint, docs-build) already
gate on `github.event_name == 'pull_request'`; no change needed.
The race the filter prevents doesn't currently bite monorel
because it's a single-module repo and the chore(release) commit
doesn't bump go.sum entries; the change is purely:
- Free CI minutes saved on chore(release) merges (the diff is
CHANGELOG + .changeset/*.md deletion + monorel internal version
string; nothing that warrants the full test matrix).
- Demonstrates the recipe in monorel's own setup as a working
reference for downstream consumers.
- Forward-compatible if monorel ever grows sub-modules.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
theogravity
enabled auto-merge (squash)
May 4, 2026 09:07
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
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.
Summary
monorel's own
ci/github/README.mddocuments a recipe telling downstream consumers to puton every job that runs
go mod tidyor otherwise resolves Go module versions. monorel's own CI didn't follow that recipe.This PR applies the filter to the three jobs in
ci.ymlthat resolve module versions (test,lint,tidy) and to thee2ejob ine2e-gitea.yml. The two PR-only jobs inci.yml(commit-lint,docs-build) already gate ongithub.event_name == 'pull_request'; no change needed.Why now
The race the filter prevents doesn't currently bite monorel because it's a single-module repo and the chore(release) commit doesn't bump
go.sumentries. Adding the filter is purely:.changeset/*.mddeletion + monorel internal version string; nothing that warrants the full test matrix or the e2e Forgejo container).Test plan
chore(release):push to main triggers a clean CI run (every filtered job runs as before).chore(release):commit's CI run showstest,lint,tidy, ande2ejobs asSkippedrather thanPass/Fail.release.ymlstill runs and tags the release; nothing else changes.🤖 Generated with Claude Code