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

[DOCS] npm 10 docs list refers to upcoming changes in node 8 #6773

Closed
2 tasks done
rotu opened this issue Sep 8, 2023 · 7 comments
Closed
2 tasks done

[DOCS] npm 10 docs list refers to upcoming changes in node 8 #6773

rotu opened this issue Sep 8, 2023 · 7 comments
Labels
Documentation documentation related issue

Comments

@rotu
Copy link
Contributor

rotu commented Sep 8, 2023

Is there an existing issue for this?

  • I have searched the existing issues

This is a CLI Docs Problem, not another kind of Docs Problem.

  • This is a CLI Docs Problem.

Description of Problem

npm 10 docs say:

A thorough re-examination of the use cases, intention, behavior, and output of this command, is currently underway. Expect significant changes to at least the default human-readable npm ls output in npm v8.

Potential Solution

No response

Affected URL

https://docs.npmjs.com/cli/v10/commands/npm-ls?v=true#note-design-changes-pending

@rotu rotu added Documentation documentation related issue Needs Triage needs review for next steps labels Sep 8, 2023
@punndcoder28
Copy link

Hi @rotu should this be removed since the current version is v10? I can work on removing the outdated documentation

@rotu
Copy link
Contributor Author

rotu commented Sep 8, 2023

Hi @rotu should this be removed since the current version is v10? I can work on removing the outdated documentation

I guess it should be removed, but probably also updated to reflect how the redesign shook out. I would guess that there were some changes since that also bear mention.

@punndcoder28
Copy link

Understood. Could you point me to the relevant changes so that I can come up with a valid suggestion for the change?

@rotu
Copy link
Contributor Author

rotu commented Sep 8, 2023

Wish I could, but I'm about as clueless as you. I would look here: https://github.com/npm/cli/blob/latest/CHANGELOG.md

Maybe @isaacs, who wrote that blurb can help.

@punndcoder28
Copy link

punndcoder28 commented Sep 9, 2023

Got it. Would going through the logical changes on the file ./lib/commands/ls.js since that commit makes sense?

@milaninfy
Copy link
Contributor

Closing: this looks to be similar if not a duplicate of an existing issue, npm/documentation#1365. If you believe your instance is unique, please provide further clarifying details.

@milaninfy milaninfy closed this as not planned Won't fix, can't repro, duplicate, stale Dec 5, 2024
@milaninfy milaninfy removed the Needs Triage needs review for next steps label Dec 5, 2024
@rotu
Copy link
Contributor Author

rotu commented Dec 5, 2024

@milaninfy The bad docs exist in this repo, so it is an issue in this repo:
https://github.com/npm/cli/blob/latest/docs/lib/content/commands/npm-ls.md

I believe this repo is the source of truth and npm/documentation#1365 should be closed, since the npm/documentation repo is derived from here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation documentation related issue
Projects
None yet
Development

No branches or pull requests

3 participants