Conversation
commit: |
There was a problem hiding this comment.
Pull request overview
This PR removes a comment from the pnpm-workspace.yaml configuration file that explains how the allowBuilds setting replaces deprecated options from older pnpm versions.
Changes:
- Removes a comment explaining that
allowBuildsreplacesonlyBuiltDependenciesandignoredBuiltDependenciesin pnpm 10.26+
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -61,7 +61,6 @@ shellEmulator: true | |||
| # Prevents Shai-Hulud-style worm attacks that exploit automatic script execution | |||
| strictDepBuilds: true | |||
|
|
|||
There was a problem hiding this comment.
The removed comment about "Replaces onlyBuiltDependencies and ignoredBuiltDependencies (pnpm 10.26+)" provides useful historical context about what the allowBuilds configuration replaces in newer pnpm versions. While this information is documented in .claude/rules/pnpm-usage.md:55, having it here helps maintainers understand the migration path from older pnpm configurations. Consider whether this context is valuable enough to retain, especially since it's directly adjacent to the allowBuilds configuration it describes.
| # allowBuilds replaces onlyBuiltDependencies and ignoredBuiltDependencies (pnpm 10.26+) |
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
Removes an unused comment from pnpm-workspace.yaml to keep the configuration file clean.
Summary by cubic
Removed an outdated comment from pnpm-workspace.yaml. Keeps the config clean and avoids confusion around allowBuilds.
Written for commit 345cc36. Summary will update on new commits.