Replies: 2 comments
-
Thanks for the update! |
Beta Was this translation helpful? Give feedback.
0 replies
-
@claudiahdz is this still the case in current |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Right now
npm ci
doesn't do any cleanup when it fails. This is hazardous because the node_modules folder might be in a broken state, but sincenpm ci
also doesn't do any staging step, the broken packages are in place and require()-able.Since
npm ci
always starts from (or creates) a clean slate, it should remove thenode_modules
folder if the installation fails.Beta Was this translation helpful? Give feedback.
All reactions