[rush] Add support for pnpm ignoredOptionalDependencies #4928
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.
Summary
Fixes #4859
Changes include updating pnpm configuration to use pnpm's
ignoredOptionalDependencies
setting. End-user should now be able to specify optional dependencies incommon/config/rush/pnpm-config.json
in case some optional dependencies are required at a later point.Details
Changes focused on updating the
rush init
template and havingignoredOptionalDependencies
incommon/temp/package.json
copied fromcommon/config/rush/pnpm-config.json
. Changes might not fully reflect until schema changes andrushVersion
are updated after merging this PR.How it was tested
rush init
(when I had the schema changed incommon/config/rush/pnpm-config.json
) to check ifcommon/config/rush/pnpm-config.json
contained newly added fieldglobalIgnoredOptionalDependencies
rush update
with local build after addingfsevents
inglobalIgnoredOptionalDependencies
to see iffsevents
was incommon/temp/package.json
Impacted documentation
PR might affect
pnpm-config.json
schema (https://rushjs.io/pages/configs/pnpm-config_json/)