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

Shouldn't publishConfig.registry just default to npm's registry if not specified? #35

Open
ghost opened this issue Feb 21, 2017 · 3 comments

Comments

@ghost
Copy link

ghost commented Feb 21, 2017

For open source users, this would be the typical default.

@jkrems
Copy link
Collaborator

jkrems commented Feb 21, 2017

I kind of like it being explicit, just in case there's ever a bug in npm or yarn that doesn't force the default to the public registry, even when using a different registry in their ~/.npmrc. I know that plenty of people were surprised that defaulting behavior exists in the first place.

But I definitely would merge a PR that removes the requirement.

@ghost
Copy link
Author

ghost commented Feb 21, 2017

True, I guess most npm users wouldn't have their own private registry, and for those that do, they've reached expert user status and should be aware of what they're doing 😁

@jkrems
Copy link
Collaborator

jkrems commented Feb 21, 2017

There are tons of people in EU/Asia that use npm proxies though or people that use a local npm cache. It's not just enterprise use cases. :)

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

1 participant