Skip to content
This repository has been archived by the owner on Aug 11, 2022. It is now read-only.

deps: Move shrinkwrap extraneous check suppression to is-extraneous #15086

Closed
wants to merge 3 commits into from

Conversation

iarna
Copy link
Contributor

@iarna iarna commented Dec 1, 2016

If your npm-shrinkwrap.json contains modules not found in your package.json we don't complain about them at install time.

We WILL complain about them at npm ls time. And we WILL complain if you try to make changes to your shrinkwrap though.

This fixes a bug where we were NOT complaining about them at npm ls time.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 85.445% when pulling 2c969d6 on iarna/fix-shrinkwrap-binding into a2a9ba7 on release-next.

If your `npm-shrinkwrap.json` contains modules not found in your
`package.json` we don't complain about them at install time.

We WILL complain about them at `npm ls` time.  And we WILL complain if you
try to make changes to your shrinkwrap though.
@iarna
Copy link
Contributor Author

iarna commented Aug 12, 2017

This is now obsolete.

@iarna iarna closed this Aug 12, 2017
@iarna iarna deleted the iarna/fix-shrinkwrap-binding branch January 5, 2018 23:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants