-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
chore(docs): update unpublish docs with both commands, removing policy info from cli docs, and added reference to unpublish policy docs #730
Conversation
nomadtechie
commented
Jan 27, 2020
- Updating docs unpublish docs to with examples of both unpublish commands
- Removing policy info from cli docs to centralize policy in policy docs
- Added reference to unpublish policy docs
…y info from cli docs, and added reference to unpublish policy docs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The small inclusion of the man pages notation.
As well as the update to the command usage (lib/unpublish.js
):
- unpublish.usage = usage('npm unpublish [<@scope>/]<pkg>[@<version>]')
+ unpublish.usage = usage(
+ 'unpublish',
+ '\nnpm unpublish [<@scope>/]<pkg>[@<version>]' +
+ '\nnpm unpublish [<@scope>/]<pkg> --force'
+ )
Co-Authored-By: Michael 'afrolion' Perrotte <mike@npmjs.com>
|
@nomadtechie noticed this got missed from our latest release & we figured out why; This PR has failing tests so it was skipped in the triage last week. Apologies for that. If we can fix whatever the issues are w/ linting/test coverage we'll pull this in next. |
lint error is resolved now - thanks again @darcyclarke ! |