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

doc: include org instructions in scoped publish #4772

Merged
merged 3 commits into from
Apr 21, 2022
Merged

Conversation

bnb
Copy link
Contributor

@bnb bnb commented Apr 18, 2022

If you've not created an organization but try to publish to a scope, you'll get a "helpful" message telling you to publish the module... that you're trying to publish. Upon reading the documentation, there's no indication that you need to actually create an organization prior to trying to publish, making it... annoying to try to publish to a net new scope.

This PR adds some documentation telling people to create an org first. There's probably more context that could be added, but wanted to get this up while it's top of mind for me so I don't continue hitting this wall without doing anything to help alleviate it.

References

@bnb bnb requested a review from a team as a code owner April 18, 2022 17:42
@bnb
Copy link
Contributor Author

bnb commented Apr 18, 2022

also made a bug report for the CLI feedback the user gets, as I don't think that feedback is a "working as intended" behaviour: #4773

@wraithgar
Copy link
Member

The scope can also be your own username. You don't have to create an org in that case.

@bnb
Copy link
Contributor Author

bnb commented Apr 18, 2022

@wraithgar updated it to reflect that and also address the possibility of being added to an existing org

Co-authored-by: Gar <wraithgar@github.com>
@lukekarrys lukekarrys merged commit fa3d829 into npm:latest Apr 21, 2022
@ruyadorno ruyadorno mentioned this pull request Apr 26, 2022
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

Successfully merging this pull request may close these issues.

3 participants