Skip to content

ci: avoid duplicate build in Node matrix - #34

Merged
rogerchappel merged 1 commit into
mainfrom
agent/oss-74901a44d914-dedupe-ci-build
Oct 9, 2026
Merged

rogerchappel merged 1 commit into
mainfrom
agent/oss-74901a44d914-dedupe-ci-build

Conversation

@rogerchappel

Copy link
Copy Markdown
Owner

Summary

  • Remove the explicit build from the Node 20/22 compatibility step; pnpm test already runs pnpm run build before template validation, so both matrix versions still build and test without repeating the build.

Verification

  • pnpm install --frozen-lockfile — passed
  • pnpm test — passed (includes TypeScript build and template registry check)
  • bash tests/cli-ux-smoke.sh — passed
  • git diff --check — passed
  • Confirmed CI matrix remains Node 20 and 22. PR/feature-branch pushes do not match the release workflow's v*.*.* tag trigger; CI's push trigger is restricted to main.

Risk Level

  • Low

Rollback Plan

  • Revert this one-line CI workflow change.

Human Decision Needed

  • None

@rogerchappel
rogerchappel marked this pull request as ready for review October 9, 2026 06:22
@rogerchappel
rogerchappel merged commit 14dec02 into main Oct 9, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant