Tooling: Set verify-deps-before-run to install instead of warn #41996
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Since upgrading to
pnpm 10
(#41821) and settingverify-deps-before-run
towarn
, we often get warnings about things being outdated:This PR switches the value to
install
.Proposed changes:
Some other options considered:
warn
: this prints toSTDOUT
instead ofSTDERR
, so pollutes command substitutionfalse
: this prevents things from being updatedprompt
: might break things in non-interactive contexts, and doesn't solve the issue if someone chooses "No"Other information:
Jetpack product discussion
Does this pull request change what data or activity we track or use?
Testing instructions: