-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[FEAT] replace preferred-pm
with package-manager-detector
#6242
Labels
enhancement
New feature or request
Milestone
Comments
Hey @benmccann looks like a good idea, would you like to work on this? |
Thanks to @benmccann, this issue is addressed in #6250 for |
emrecancorapci
pushed a commit
to emrecancorapci/refine
that referenced
this issue
Sep 4, 2024
emrecancorapci
pushed a commit
to emrecancorapci/refine
that referenced
this issue
Sep 4, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is your feature request related to a problem? Please describe.
preferred-pm pulls in 24 dependencies, which is unnecessary for what it does: https://npmgraph.js.org/?q=preferred-pm
package-manager-detector has no dependencies: https://npmgraph.js.org/?q=package-manager-detector
Describe alternatives you've considered
No response
Additional context
This code has been battle-tested for a long time in the heavily used
@antfu/ni
. It was recently extracted out into its own library. I am switching Svelte's CLI to use it: svelte-add/svelte-add#535Describe the thing to improve
Reduce the number of dependencies
The text was updated successfully, but these errors were encountered: