Skip to content

Commit

Permalink
feat: notify on adduser of upcoming cmds, login and register
Browse files Browse the repository at this point in the history
  • Loading branch information
fritzy authored and wraithgar committed Jul 12, 2022
1 parent 52ec5ec commit c6c4ba3
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
4 changes: 4 additions & 0 deletions lib/commands/adduser.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,10 @@ class AddUser extends BaseCommand {

log.disableProgress()

log.warn('adduser',
'`adduser` will be split into `login` and `register in a future version.'
+ ' `adduser` will become an alias of `register`.'
+ ' `login` (currently an alias) will become its own command.')
log.notice('', `Log in on ${replaceInfo(registry)}`)

const { message, newCreds } = await auth(this.npm, {
Expand Down
6 changes: 1 addition & 5 deletions lib/utils/config/definitions.js
Original file line number Diff line number Diff line change
Expand Up @@ -239,12 +239,8 @@ define('audit-level', {
define('auth-type', {
default: 'legacy',
type: ['legacy', 'webauthn', 'sso', 'saml', 'oauth'],
deprecated: `
The SSO/SAML/OAuth methods are deprecated and will be removed in
a future version of npm in favor of web-based login.
`,
description: `
What authentication strategy to use with \`adduser\`/\`login\`.
What authentication strategy to use with \`login\`.
Pass \`webauthn\` to use a web-based login.
`,
Expand Down

0 comments on commit c6c4ba3

Please sign in to comment.