Skip to content

build: set verifyDepsBeforeRun to warn in pnpm workspace configuration - #34071

Merged
alan-agius4 merged 1 commit into
angular:mainfrom
alan-agius4:build/pnpm-verify-deps-warn
Sep 10, 2026
Merged

build: set verifyDepsBeforeRun to warn in pnpm workspace configuration#34071
alan-agius4 merged 1 commit into
angular:mainfrom
alan-agius4:build/pnpm-verify-deps-warn

Conversation

@alan-agius4

Copy link
Copy Markdown
Collaborator

The default 'install' setting causes unexpected lockfile updates during pnpm ng-dev pr merge. Setting it to warn prevents automatic dependency installation while still alerting if dependencies are out of sync.

The default 'install' setting causes unexpected lockfile updates during `pnpm ng-dev pr merge`. Setting it to `warn` prevents automatic dependency installation while still alerting if dependencies are out of sync.
@angular-robot angular-robot Bot added the area: build & ci Related the build and CI infrastructure of the project label Sep 10, 2026
@alan-agius4 alan-agius4 added target: patch This PR is targeted for the next patch release action: merge The PR is ready for merge by the caretaker labels Sep 10, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds a workaround to avoid lockfile updates during PR merges by setting verifyDepsBeforeRun: warn in pnpm-workspace.yaml. However, this configuration option is not supported in pnpm-workspace.yaml and must be moved to the root .npmrc file as verify-deps-before-run=warn to be effective.

Comment thread pnpm-workspace.yaml
@alan-agius4
alan-agius4 merged commit af7f00e into angular:main Sep 10, 2026
18 checks passed
@alan-agius4
alan-agius4 deleted the build/pnpm-verify-deps-warn branch September 10, 2026 14:37
@alan-agius4

Copy link
Copy Markdown
Collaborator Author

This PR was merged into the repository. The changes were merged into the following branches:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants