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 cidoesn't do any cleanup when it fails. This is hazardous because the node_modules folder might be in a broken state, but sincenpm cialso doesn't do any staging step, the broken packages are in place and require()-able.Since
npm cialways starts from (or creates) a clean slate, it should remove thenode_modulesfolder if the installation fails.Beta Was this translation helpful? Give feedback.
All reactions