-
-
Notifications
You must be signed in to change notification settings - Fork 226
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
False positives produced by deduping #126
Comments
I don't understand how false happens on this case. Does the |
Yes, it moves the deduped packages into |
OK, is there any extra information to indicate which dedup package is used by which package? Maybe I should try soon and revisit here. |
I run |
Yes I'm running npm 2. Here's a sample package.json: {
"dependencies": {
"mime-kind": "1.0.4",
"nodal": "0.8.3",
"octocat": "0.11.0"
}
} After dedupe: {
"dependencies": {
"ansi-regex": "2.0.0",
"async": "1.5.2",
"chalk": "1.1.1",
"lodash": "3.10.1",
"mime-kind": "1.0.4",
"nodal": "0.8.3",
"octocat": "0.11.0",
"strip-ansi": "3.0.1"
}
} |
I get it. This is do-able. However, I want to ask the first simple question, why not use npm@3? |
npm 3 is considerably slower and I have had some compatibility issues |
OK. I will implement this after the current version 0.6.2 is out. Mark it on 0.6.3. |
While working on detected dedup packages, I have some questions:
|
Newer npm versions no longer implement this style of deduping (since a while, now), so I'm going to close this issue as stale. If anyone encounters a similar issue, please feel free to open a new issue! |
False positives are reported when having done a dedupe. Would be nice to have an option to exclude deduped packages.
The text was updated successfully, but these errors were encountered: