-
Notifications
You must be signed in to change notification settings - Fork 530
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
npm-aliases: handle aliases for package-lock.json #1349
Conversation
This PR adds support for npm lockfile version 3, which drops the "dependencies" key and uses "packages" instead. I've refactored the lockfile parser to make the distinction between the versions explicit rather than the implicit behaviour before. It _might_ be worth splitting into separate files at some point, but the logic is so minimal that I haven't done it. Fixes #1203 Signed-off-by: Rob Cresswell <robcresswell@users.noreply.github.com>
a914fd8
to
0bd18b4
Compare
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.
Thanks @Mikcl this looks great, and thanks for adding tests 👍
It looks like there are some checks failing in lint & tests
Hi @Mikcl could you |
Signed-off-by: mikcl <mikesmikes400@gmail.com>
86f3957
to
437e267
Compare
Fixes #1314
This only addresses npm/
package-lock.json
aliases.Addresses package-lock.json version 1-3.
the
package.json
that thepackage-lock.json
s examples were derived from:https://gist.github.com/Mikcl/aef4d180f4275b66252dca32b6668375#file-package-json