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

npm ls --production #2853

Closed
vvo opened this issue Oct 4, 2012 · 4 comments
Closed

npm ls --production #2853

vvo opened this issue Oct 4, 2012 · 4 comments

Comments

@vvo
Copy link

vvo commented Oct 4, 2012

Hello, could npm ls works with --production --dev flags?

@isaacs
Copy link
Contributor

isaacs commented Oct 8, 2012

+1.

@vvo
Copy link
Author

vvo commented Oct 18, 2012

After reading npm ls code, and isaacs/read-installed, it seems to me I should add a fourth argument to readInstalled:

function readInstalled (folder, depth, env, cb) {
...
}

Because readInstalled does not have access to npm.config.get("production").
And when doing npm ls --production it does not trigger NODE_ENV=production.

Or I could re-read (already done in readInstalled), package.json and filter readInstalled on devDependencies.

  1. Fourth argument to readInstalled?
  2. Read package.json from npm ls and filter?

@watilde
Copy link
Contributor

watilde commented Mar 8, 2015

Maybe this issue can close by #7413

@vvo
Copy link
Author

vvo commented Jan 8, 2016

yep

@vvo vvo closed this as completed Jan 8, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants