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

Add feature to show dependencies of a particular dependency #22

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

alanyinjs
Copy link

No description provided.

Copy link
Contributor

@zkat zkat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I really like this feature, but I don't feel great about the interface. I'd like to see some exploration of alternative interfaces for this that are clearer -- preferably not using --depth because we don't like that flag to begin with. Thanks for your time submitting this! And let me know if you have any questions. I think filling out "alternatives" with a bit of brainstorming would push this far.

@zkat
Copy link
Contributor

zkat commented Jan 31, 2019

Hey, touching base here. Are you still interested in this RFC?

@alanyinjs
Copy link
Author

@zkat Yes I'm still interested! Just trying to come up with an alternative...

@leckylao
Copy link

In that case let's remove the depth and just pass in the package to start with. That way can still beneficial from showing dependency for a specific package.

@darcyclarke darcyclarke added Agenda will be discussed at the Open RFC call Enhancement new feature or improvement Help Wanted Needs Documentation pull request requires docs before merging semver:minor new backwards-compatible feature labels Oct 30, 2019
@darcyclarke darcyclarke added Release 7.x and removed Agenda will be discussed at the Open RFC call Help Wanted Needs Documentation pull request requires docs before merging semver:minor new backwards-compatible feature labels Nov 27, 2019
@mikemimik
Copy link

The root problem that you're trying to solve for is to filter the view of the dependencies in your project. This is something we very much want as a feature in npm/cli project. However, we don't be able to accomplish this until npm@7 due to some performance, and timeline restrictions. We're definitely going to revisit this once we have the beta of npm@7 out in the publics hands. Thanks so much @alanyinjs for creating this RFC. We'll circle back in Q1 of 2020 on this to get it implemented in npm@7 👍

@settings settings bot removed the Release 7.x label Sep 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement new feature or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants