Skip to content

ci(release): gate publish on inline lint/typecheck/test #18

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ylabonte merged 1 commit into from
May 18, 2026
Merged

ci(release): gate publish on inline lint/typecheck/test #18

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 13 additions & 0 deletions .github/workflows/release.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,19 @@ jobs:
# npm 11.5+ is required for trusted-publishing OIDC. Node 20 ships with npm 10.
- run: npm install -g npm@latest
- run: pnpm install --frozen-lockfile
# Inline CI gate: re-run the same checks the CI workflow does on push to
# main. CI and Release fire in parallel on `push: main`, so without an
# explicit gate here a merge that broke lint/typecheck/test could still
# trigger publish (changesets/action would happily run `pnpm release` on
# a red tree, and a regression that doesn't crash `pnpm build` would
# ship). The four checks below mirror CLAUDE.md's pre-commit discipline
# and the steps in `ci.yml`; duplication is cheap on a warm pnpm cache
# (~30s total) and the alternative (workflow_run trigger, or merging
# the workflows) trades that off against worse separation of concerns.
- run: pnpm format:check
- run: pnpm lint
- run: pnpm typecheck
- run: pnpm test
- run: pnpm build
- name: Create release PR or publish
id: changesets
Expand Down
Loading