Skip to content

fix: let pnpm run dep build scripts (ERR_PNPM_IGNORED_BUILDS)#152

Merged
JumpLink merged 1 commit into
mainfrom
fix/pnpm-allow-builds
Jun 22, 2026
Merged

fix: let pnpm run dep build scripts (ERR_PNPM_IGNORED_BUILDS)#152
JumpLink merged 1 commit into
mainfrom
fix/pnpm-allow-builds

Conversation

@JumpLink

Copy link
Copy Markdown
Owner

Follow-up to #150/#151. The Package Managers workflow's pnpm leg now gets past the 404 + age gate (workspaces link) but exits 1 on ERR_PNPM_IGNORED_BUILDS: pnpm 10 blocks dependency build scripts by default and errors non-interactively. npm and yarn run dep build scripts by default (hence their green legs), so adding dangerouslyAllowAllBuilds: true to pnpm-workspace.yaml makes pnpm match them — identical, already-trusted dep set.

Verified in a clean HOME (mimics CI): pnpm install exits 0; the workflow's check (build @learn6502/6502 → check @learn6502/common-ui) exits 0. Only touches pnpm-workspace.yaml + the README note.

After the previous pnpm fix, the Package Managers workflow's pnpm leg got past
the 404 + age gate (workspaces now link) but exited 1 on ERR_PNPM_IGNORED_BUILDS:
pnpm 10 blocks dependency build scripts by default and errors in
CI/non-interactive runs. npm and yarn RUN dep build scripts by default (which is
why their legs are green), so add `dangerouslyAllowAllBuilds: true` to make pnpm
match them — same trusted dep set, just no pnpm-specific opt-in friction.

Verified in a clean HOME (mimicking CI): `pnpm install` exits 0, and the
workflow's check (build @learn6502/6502 → check @learn6502/common-ui) exits 0.
@JumpLink JumpLink merged commit d89deda into main Jun 22, 2026
3 checks passed
@JumpLink JumpLink deleted the fix/pnpm-allow-builds branch June 22, 2026 17:31
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