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
pnpm@10.9.0also accept this syntax and convert it into json-syntax using sideEffectsCache.When the above command is executed on a empty
pnpm@10.9.0project it writes now to the filepnpm-workspace.yamlinstead 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
.npmrcand applyto update the corresponding
pnpm-workspace.yamlfile withsideEffectsCache: false