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

ls: Don't exclude modules that are both dev & prod with --production #15088

Closed

Conversation

iarna
Copy link
Contributor

@iarna iarna commented Dec 1, 2016

Previously if a module was in devDependencies then it would never show up with npm ls --production even if it was ALSO in dependencies.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 85.461% when pulling 9a6204f on iarna/ls-do-not-skip-multi-mode-deps-with-prod into a2a9ba7 on release-next.

@zkat zkat force-pushed the release-next branch 4 times, most recently from e1c7ae8 to 04d7c2f Compare December 2, 2016 01:05
@iarna iarna force-pushed the release-next branch 7 times, most recently from 20e7365 to ff1f0bb Compare December 16, 2016 01:53
@zkat zkat force-pushed the release-next branch 2 times, most recently from 692ff0e to 892f531 Compare February 10, 2017 19:02
@iarna iarna force-pushed the release-next branch 5 times, most recently from 6465f26 to d841f73 Compare February 24, 2017 01:04
Previously if a module was in devDependencies then it would never show
up with `npm ls --production` even if it was ALSO in dependencies.
@iarna iarna force-pushed the iarna/ls-do-not-skip-multi-mode-deps-with-prod branch from 9a6204f to cda7f9e Compare March 8, 2017 00:39
@zkat zkat force-pushed the release-next branch 2 times, most recently from 7bd08ba to 550efed Compare March 10, 2017 00:54
@iarna
Copy link
Contributor Author

iarna commented Aug 13, 2017

merged as 7a7710b

@iarna iarna closed this Aug 13, 2017
iarna added a commit that referenced this pull request Aug 13, 2017
Previously if a module was in devDependencies then it would never show
up with `npm ls --production` even if it was ALSO in dependencies.

Credit: @iarna
PR-URL: #15088
@iarna iarna deleted the iarna/ls-do-not-skip-multi-mode-deps-with-prod branch January 6, 2018 00:13
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.

2 participants