-
Notifications
You must be signed in to change notification settings - Fork 239
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
[RRFC] Deprecating the npx package from the public registry #238
Comments
I have heard this a few times, but I am pretty strongly against this idea. It is now deeply ingrained in many folks workflows. Wouldn't it be better to just re-purpose the package on top of whatever new approach you prefer to maintain? |
@wesleytodd awesome to hear from you! Yeah, I was expecting that there could be some concerns so I think it's a great point to talk about during the OpenRFC call and make sure we are aware of all the implications! 😊 |
Is there any additional context available here or discussion to follow on this topic? In particular understanding the motivation to deprecate it and what the alternative being proposed is (if any)? I (personally) think
|
hi @thescientist13 thanks for the comment, it just made me realize some people might not be aware of all the context so I added a big disclaimer there on top:
|
AH, phew! 😅 Appreciate the clarification there. 👍 |
How many people actually use/install the npx package from the registry, since npx comes with modern versions of npm? |
@ljharb ~56k weekly downloads |
@darcyclarke do you have any internal metrics that tell you if those are being installed by an npm that already ships with npx? ie, can you filter out the installs that arguably aren't needed? |
Just to put in perspective: 56k weekly downloads are about 2.75% of the 2 million+ weekly downloads the Maybe something like: "npx is now integrated directly into npm v7+. There are numerous unsupported issues in this package that are fixed in the latest npm version." |
These are long tail migrations. What are the benefits of adding a deprecation warning (which will be disruptive)? Seems to me there should be some big benefits to justify it until npm 7 has seen significant adoption. So share some data, of the projects I am able to reliably query at work, 12% of active projects are still not on npm@6. EDIT: and to be clear, I am not saying this would disrupt those projects, just that saying "it is in npm@7" doesn't solve things for everyone. |
Another point is that if you don't want to force your consumers to have npm 7, or even an npm with npx, you'd want to add npx as a dependency. |
Follow up from the RFC discussions, add a deprecation notice to the README as a first step into deprecating/archiving this repo. Ref: npm/rfcs#238 Ref: https://github.com/npm/rfcs/blob/cbf076b7a58be9d39d12bd8b91c8cf89612bf8c3/meetings/2020-10-14.md
Follow up from the RFC discussions, add a deprecation notice to the README as a first step into deprecating/archiving this repo. Ref: npm/rfcs#238 Ref: https://github.com/npm/rfcs/blob/cbf076b7a58be9d39d12bd8b91c8cf89612bf8c3/meetings/2020-10-14.md
We know we want to deprecate the npx package but what exactly does that mean?
The text was updated successfully, but these errors were encountered: