test: move side-effects-cache=false for pnpm #1435
Merged
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.
Issue
In order to disable the pnpm side effects cache, to prevent pnpm from skipping Cypress binary installation if the Cypress npm module is already cached, the Cypress documentation > pnpm Configuration now recommends using:
This uses ini-syntax and is compatible with pnpm 9.x and earlier 10.x versions. Later versions, such as
[email protected]
also accept this syntax and convert it into json-syntax using sideEffectsCache.When the above command is executed on a empty
[email protected]
project it writes now to the filepnpm-workspace.yaml
instead of to.npmrc
.The pnpm bookmark https://pnpm.io/npmrc#side-effects-cache no longer exists. It has been moved to https://pnpm.io/settings#sideeffectscache as part of https://pnpm.io/settings#build-settings.
Change
In the README.md > pnpm document section, link to https://on.cypress.io/install#pnpm-Configuration so that pnpm configuration instructions are not repeated and can be maintained in one place, on the Cypress documentation site for general use.
In
delete
.npmrc
and applyto update the corresponding
pnpm-workspace.yaml
file withsideEffectsCache: false