Skip to content

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

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

fix(config): align .npmrc and pnpm-workspace.yaml for pnpm v11#62
John-David Dalton (jdalton) merged 2 commits intomainfrom
fix/pnpm-workspace-config

Conversation

@jdalton
Copy link
Copy Markdown
Collaborator

Summary

  • Remove pnpm settings from .npmrc (pnpm v11 only reads auth/registry from .npmrc)
  • Add trustPolicy to pnpm-workspace.yaml (migrated from .npmrc)
  • Remove invalid ignoreDependencyScripts and linkWorkspacePackages settings
  • Keep ignore-scripts and min-release-age in .npmrc for npm compatibility

Test plan

  • Verify pnpm install works locally
  • Verify CI passes

Use remote default branch as baseline instead of release tags.
Prevents false positives from scanning commits already on main.
- .npmrc: keep only npm-valid settings (ignore-scripts, min-release-age)
- .npmrc: remove trust-policy (not a valid npm setting)
- pnpm-workspace.yaml: remove ignoreDependencyScripts (invalid setting name)
- pnpm-workspace.yaml: remove linkWorkspacePackages (removed in pnpm v11)
- pnpm-workspace.yaml: add trustPolicy (pnpm equivalent)
- Rely on pnpm v11 strictDepBuilds (default true) + allowBuilds for dep scripts
@jdalton John-David Dalton (jdalton) merged commit 6eb1dae into main Apr 14, 2026
9 checks passed
@jdalton John-David Dalton (jdalton) deleted the fix/pnpm-workspace-config branch April 14, 2026 13:17
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