Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert changes introduced in #990 #994

Merged
merged 1 commit into from
Mar 30, 2022

Conversation

colincasey
Copy link
Contributor

@colincasey colincasey commented Mar 30, 2022

There is an issue with the changes to prune the yarn 2 cache that, in certain situations, would cause all devDependencies to be reinstalled during the prune stage which is the opposite result of the intended fix. This can cause slug sizes to increase for customers so this PR restores the previous cache clearing behavior.

Reverts #918, GUS-W-9502576

There is an issue with the changes to prune the yarn 2 cache that, in certain situations, would cause all devDependencies to be reinstalled during the prune stage which is the opposite result of the intended fix.  This can cause slug sizes to increase for customers so this PR restores the previous cache clearing behavior.
@colincasey colincasey requested a review from a team as a code owner March 30, 2022 19:27
@colincasey colincasey merged commit 66aa7b0 into main Mar 30, 2022
@colincasey colincasey deleted the revert/v193_yarn_dependency_pruning branch March 30, 2022 20:38
@codingjoe
Copy link

@colincasey @joshwlewis I think this just broke our deployment.

-----> Installing dependencies
       Installing node modules
       npm ERR! `npm ci` can only install packages when your package.json and package-lock.json or npm-shrinkwrap.json are in sync. Please update your lock file with `npm install` before continuing.
       npm ERR! 
       npm ERR! Invalid: lock file's [email protected] does not satisfy [email protected]
       npm ERR! 
       
       npm ERR! A complete log of this run can be found in:
       npm ERR!     /tmp/npmcache.ZyR7S/_logs/2022-04-02T19_04_38_141Z-debug-0.log

Both files are present and unchanged in the git repo from which we deploy this app.

@colincasey
Copy link
Contributor Author

@codingjoe this PR was to address how Yarn 2 dependencies get pruned. It should't affect npm installs.

Does the issue go away if you follow the directions in the npm ci error output?

  • run npm install locally
  • commit the changes to your package-lock.json

@codingjoe
Copy link

We further investigated this. We didn't change the package file or the lock. However, there was a change on NPMs package index. Peculiar error, sorry for the false alarm.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants