Skip to content

fix(config): align .npmrc and pnpm-workspace.yaml for pnpm v11#267

Merged
John-David Dalton (jdalton) merged 1 commit intomainfrom
fix/pnpm-workspace-config
Apr 14, 2026
Merged

fix(config): align .npmrc and pnpm-workspace.yaml for pnpm v11#267
John-David Dalton (jdalton) merged 1 commit intomainfrom
fix/pnpm-workspace-config

Conversation

@jdalton
Copy link
Copy Markdown
Collaborator

@jdalton John-David Dalton (jdalton) commented Apr 14, 2026

Summary

  • Remove ignoreScripts: true from pnpm-workspace.yaml (was blocking project prepare scripts)
  • Remove linkWorkspacePackages (removed in pnpm v11)
  • Add ignore-scripts=true to .npmrc for npm compatibility
  • Rely on pnpm v11 strictDepBuilds (default true) + allowBuilds for dep script control

Test plan

  • Verify pnpm install works locally
  • Verify CI passes

- .npmrc: add ignore-scripts (valid npm setting, safety net)
- pnpm-workspace.yaml: remove ignoreScripts (blocks project prepare/build scripts)
- pnpm-workspace.yaml: remove linkWorkspacePackages (removed in pnpm v11)
- Rely on pnpm v11 strictDepBuilds (default true) + allowBuilds for dep scripts
@jdalton John-David Dalton (jdalton) changed the title fix(config): add resolutionMode highest to pnpm-workspace.yaml fix(config): align .npmrc and pnpm-workspace.yaml for pnpm v11 Apr 14, 2026
@jdalton John-David Dalton (jdalton) merged commit 4377ac9 into main Apr 14, 2026
14 checks passed
@jdalton John-David Dalton (jdalton) deleted the fix/pnpm-workspace-config branch April 14, 2026 13:15
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