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
feat: use Arborist for dedupe and prune #1395
Conversation
4b02ef0
to
56d458e
Compare
@isaacs! While adding dryRun for I could no longer get the aliased command from |
As discussed with Isaac, |
@@ -60,6 +60,8 @@ Modules | |||
Note that this operation transforms the dependency tree, but will never | |||
result in new modules being installed. | |||
|
|||
Using `npm find-dupes` will run the command in dryRun mode. | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh if npm find-dupes
is now a top-level command it'd be nice to also add docs for it and a link below in the See Also section of this file 👇
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure about creating a new docs file for this command since it is essentially the same as dedupe
only on dryRun mode. What do you think about just pointing the find-dupes
docs to the dedupe
docs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
right! sounds good 👍
PR-URL: #1395 Credit: @claudiahdz Close: #1395 Reviewed-by: @ruyadorno
merged 😄 |
This PR uses Arborist for dedupe and prune.
Dedupe docs have been updated to include the fact that
npm find-dupes
runs it in dryRun mode (which didn't seem to be documented anywhere).