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

Yarn support? #2

Closed
nicolo-ribaudo opened this issue Aug 17, 2018 · 8 comments
Closed

Yarn support? #2

nicolo-ribaudo opened this issue Aug 17, 2018 · 8 comments

Comments

@nicolo-ribaudo
Copy link

The output of this package is much more nicer than the yarn output:

➜ npx npm-why @babel/helpers
npx: installato 10 in 0.852s

  Who required @babel/helpers:

  babel-upgrade > @babel/core > @babel/helpers@7.0.0-beta.41

➜ yarn why babel-core
yarn why v1.9.4
[1/4] Why do we have the module "babel-core"...?
[2/4] Initialising dependency graph...
[3/4] Finding dependency...
[4/4] Calculating file sizes...
=> Found "jest-config#babel-core@6.26.0"
info This module exists because "jest#jest-cli#jest-config" depends on it.
info Disk size without dependencies: "228KB"
info Disk size with unique dependencies: "33.28MB"
info Disk size with transitive dependencies: "41.84MB"
info Number of shared dependencies: 40

I understand that this package was created specifically for npm because yarn already supported it, but what do you think of supporting both the package managers?

@amio
Copy link
Owner

amio commented Aug 17, 2018

Glad you like it :D

I'm not sure if it's viable, yarn.lock stores less information than package.json. I'll check on this later.

BTW, if it‘s viable, I think this feature should be under a new name other than npm-why 😄

@nicolo-ribaudo
Copy link
Author

BTW, if it‘s viable, I think this feature should be under a new name other than npm-why 😄

Well, having them in the same package would be much more handy, so I don't have to ask users to run different commands based on their package manager 🙂
e.g. babel/babel#8482 (comment)

@amio
Copy link
Owner

amio commented Aug 17, 2018

I was thinking why-pkg (depend on npm-why and yarn-why) to do the handy job.

@amio
Copy link
Owner

amio commented Aug 21, 2018

@nicolo-ribaudo yarn-why is live now, npx yarn-why @babel/helpers see if it works as expected.

@nicolo-ribaudo
Copy link
Author

It works, thank you! 😄

@amio
Copy link
Owner

amio commented Aug 23, 2018

@nicolo-ribaudo and the yarn & npm support are under nls now: npx nls why @babel/helpers.

@amio amio closed this as completed Sep 27, 2018
@nicolo-ribaudo
Copy link
Author

@amio Thank you again, npx nls why is making it a lot easier to help users

@amio
Copy link
Owner

amio commented Sep 28, 2018

@nicolo-ribaudo Glad to be helpful 😄

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

No branches or pull requests

2 participants