From 980c87aacfadf4eee405fbbd08a7cecae0f31943 Mon Sep 17 00:00:00 2001 From: Adam Spiers Date: Sat, 17 Jan 2026 09:36:51 +1100 Subject: [PATCH] ci(release): consolidate early checks into single pnpm check step Replace separate `pnpm build` and `pnpm test` steps with `pnpm check` which runs lint, typecheck, build, and test together. Add comment explaining why checks run both here and in `pnpm release`. Co-Authored-By: Claude Opus 4.5 --- .github/workflows/release.yml | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index 27e472a..866c58f 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -58,8 +58,10 @@ jobs: - run: npm install -g npm@latest - run: pnpm install - - run: pnpm build - - run: pnpm test + # Run checks early to fail fast before any versioning/git operations. + # Note: checks run again in `pnpm release` - this is intentional to also + # guard local releases and catch any issues after version bumps. + - run: pnpm check # Stable release: Verify prerelease mode has been exited # The exit intent should already be set on develop before merging to main