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

Documentation missing for npm login #1880

Closed
jameshfisher opened this issue Sep 30, 2020 · 5 comments
Closed

Documentation missing for npm login #1880

jameshfisher opened this issue Sep 30, 2020 · 5 comments
Labels
Enhancement new feature or improvement Release 7.x work is associated with a specific npm 7 release

Comments

@jameshfisher
Copy link

Current Behavior:

https://docs.npmjs.com/cli-documentation/cli lists commands sans login

Expected Behavior:

Page lists all commands listed by:

$ npm --help

Usage: npm <command>

where <command> is one of:
    access, adduser, audit, bin, bugs, c, cache, ci, cit,
    clean-install, clean-install-test, completion, config,
    create, ddp, dedupe, deprecate, dist-tag, docs, doctor,
    edit, explore, fund, get, help, help-search, hook, i, init,
    install, install-ci-test, install-test, it, link, list, ln,
    login, logout, ls, org, outdated, owner, pack, ping, prefix,
    profile, prune, publish, rb, rebuild, repo, restart, root,
    run, run-script, s, se, search, set, shrinkwrap, star,
    stars, start, stop, t, team, test, token, tst, un,
    uninstall, unpublish, unstar, up, update, v, version, view,
    whoami

Steps To Reproduce:

Visit https://docs.npmjs.com/cli-documentation/cli

@jameshfisher jameshfisher added Bug thing that needs fixing Needs Triage needs review for next steps Release 7.x work is associated with a specific npm 7 release labels Sep 30, 2020
@darcyclarke
Copy link
Contributor

Hey @jamesgeorge007, unfortunately, npm login is actually just an alias of npm adduser so the docs do indeed exist. We'll keep this open & make it a point to add a duplicate ref to the existing doc/info.

@darcyclarke darcyclarke added Enhancement new feature or improvement and removed Bug thing that needs fixing Needs Triage needs review for next steps beta labels Oct 1, 2020
@jameshfisher
Copy link
Author

Thanks @darcyclarke - weird, adduser sounds like "sign up" rather than "log in", but OK!

@ljharb
Copy link
Contributor

ljharb commented Oct 1, 2020

@darcyclarke tbh it's always seemed weird to me that "login" isn't the primary command, "adduser" is weird. Could the relationship be switched?

@Celtian
Copy link

Celtian commented Nov 7, 2020

Can you add also here
https://docs.npmjs.com/cli/v6/commands/npm-unpublish
information that you should run first command "npm login" before you try to unpublish something. If you are not logged cli gives me irrelevant 404 error.

wraithgar added a commit that referenced this issue Jan 11, 2021
Mostly grammar fixes, also adds not about needing to
be logged in, as per comment in #1880
wraithgar added a commit that referenced this issue Jan 14, 2021
Mostly grammar fixes, also adds not about needing to
be logged in, as per comment in #1880

PR-URL: #2474
Credit: @wraithgar
Close: #2474
Reviewed-by: @darcyclarke
@wraithgar
Copy link
Member

This was fixed in npm 8, "login" and "adduser" were separated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement new feature or improvement Release 7.x work is associated with a specific npm 7 release
Projects
None yet
Development

No branches or pull requests

5 participants