Skip to content

Fix pnpm-workspace.yaml settings nesting#691

Merged
justinnais merged 1 commit intomasterfrom
ACQ-6300-fix-pnpm-workspace-settings
Feb 12, 2026
Merged

Fix pnpm-workspace.yaml settings nesting#691
justinnais merged 1 commit intomasterfrom
ACQ-6300-fix-pnpm-workspace-settings

Conversation

@justinnais
Copy link
Contributor

@justinnais justinnais commented Feb 12, 2026

Summary

  • Remove invalid settings: nesting from pnpm-workspace.yaml
  • Remove unnecessary autoInstallPeers: true configuration
  • Remove unnecessary strictPeerDependencies: false configuration (default)
  • Keep only minimumReleaseAge and minimumReleaseAgeExclude at root level

This aligns with the pnpm workspace specification and matches the configuration used in the web repository.

@justinnais justinnais requested a review from a team as a code owner February 12, 2026 00:17
@justinnais justinnais requested a review from mcinbell February 12, 2026 00:17
@justinnais justinnais marked this pull request as draft February 12, 2026 00:29
@justinnais justinnais force-pushed the ACQ-6300-fix-pnpm-workspace-settings branch from 371a320 to 47287e5 Compare February 12, 2026 00:39
Remove invalid 'settings:' nesting and unnecessary configuration options. The 'autoInstallPeers' and 'strictPeerDependencies' settings are not valid configuration options for pnpm-workspace.yaml and belong in .npmrc instead. Keep only the valid minimumReleaseAge security setting and add minimumReleaseAgeExclude for internal packages.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@justinnais justinnais force-pushed the ACQ-6300-fix-pnpm-workspace-settings branch from 47287e5 to 9bd918a Compare February 12, 2026 01:06
@justinnais justinnais marked this pull request as ready for review February 12, 2026 01:06
@justinnais justinnais requested a review from tory24 February 12, 2026 01:07
@justinnais justinnais merged commit 28fc931 into master Feb 12, 2026
9 checks passed
@justinnais justinnais deleted the ACQ-6300-fix-pnpm-workspace-settings branch February 12, 2026 01:14
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.

2 participants