Skip to content
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

Indicate top-level dep, or transitive via who #30

Closed
ljharb opened this issue Feb 14, 2019 · 4 comments
Closed

Indicate top-level dep, or transitive via who #30

ljharb opened this issue Feb 14, 2019 · 4 comments

Comments

@ljharb
Copy link
Member

ljharb commented Feb 14, 2019

It'd be awesome if the report indicated whether a dep was top-level, or if not, which packages caused it to be included.

@kemitchell
Copy link
Member

@ljharb I suspect read-package-tree includes that information in records. Perhaps "encoded" in the module's path.

The place to bring that in is probably

var result = {

The place to print it out is probably

function toText (result) {

I'm a bit slammed. Feel free to send a PR. npm run style lints.

@kemitchell
Copy link
Member

Could we just print path?

@ljharb
Copy link
Member Author

ljharb commented Feb 16, 2019

Dep hoisting makes the path unhelpful.

@kemitchell
Copy link
Member

Yeah, this is going to get nasty.

If you'd like to send a PR, by all means do. But this isn't something I'm going to tackle.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants