Add a standalone script suitable for cleaning node_modules #138
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This is a workaround/solution for annoying problems: * Unlike almost every other package, I would really really like to be able to use rimraf from package scripts prior to a successful installation/population of node_modules * I would like to be able to use rimraf to delete node_modules itself, in its entirety, on Windows, without relying on a global rimraf installation
isaacs
reviewed
Mar 2, 2017
| directory. Unfortunately, this has two major downsides: | ||
|
|
||
| * Must `npm install` to have rimraf available for cleaning. | ||
| * On some platforms (possibland possibly other platforms) this fails, |
|
Yeah, this feels like it should be a separate module. I'm happy to provide a link in the readme, once you publish it. It's a bit outside of the contract of rimraf, and only useful for a very specific use case (deleting the folder where rimraf is installed) and only relevant to Windows. Just out of curiosity, how often do you find it necessary to fully delete |
|
Thank you, I will come up with a variation that makes sense to publish only on its own - without the base rimraf functionality even. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This is a workaround/solution for annoying problems:
Unlike almost every other package, I would really really like to be able to use rimraf from package scripts prior to a successful installation/population of node_modules
I would like to be able to use rimraf to delete node_modules itself, in its entirety, on Windows, without relying on a global rimraf installation
I think it fixes #102.
It may make more sense for me to just publish this somewhere else, @isaacs please feel free to unceremoniously close my PR if you disagree with the whole approach taken here!