search

sindresorhus / np

A better `npm publish`
MIT License
7.56k stars 299 forks source link

Improvements and fixes #721

Closed mifi closed 1 year ago

mifi commented 1 year ago

This PR fixes a few bugs and allow publishing scoped packages (to avoid 402 error). Actually these are all bugs I discovered while working on support for yarn berry in np.

I realize that you're not interested in supporting yarn berry #612 - however I use it a lot and I use np as well, so I figured I'd try to add support for it. I can submit a separate PR for berry support, but if you're still not interested I'll publish a fork of np instead. Here's the branch: https://github.com/mifi/np/pull/1

for easier review, see each commit separately.

sindresorhus commented 1 year ago

I can submit a separate PR for berry support

Sorry, no interest. I don't want more code-paths to maintain. Everyone wants their feature supported, but no one stays around in the long-term to fix bugs.

mifi commented 1 year ago

I have now reverted the --publish-scoped option added a note to the readme, thanks!

sindresorhus commented 1 year ago

Thanks :)