Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Aug 11, 2022. It is now read-only.

Conversation

happylynx
Copy link
Contributor

-l option of npm config list documented.

Previously it was only mentioned in output of npm config list:

; "npm config ls -l" to show all defaults.

@isaacs isaacs added the review label May 28, 2017
Copy link
Contributor

@zkat zkat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a tiny nit, but this looks good. Thanks for taking care of it!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would probably reword this a bit to something like: Use -l to also show defaults.

@zkat zkat changed the base branch from master to release-next May 31, 2017 06:08
`-l` option of `npm config list` documented.

Previously it was only mentioned in output of `npm config list`:

    ; "npm config ls -l" to show all defaults.
Copy link
Contributor

@zkat zkat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks 💃

@zkat zkat merged commit 69e2522 into npm:release-next Jun 29, 2017
zkat pushed a commit that referenced this pull request Jul 5, 2017
`-l` option of `npm config list` documented.

Previously it was only mentioned in output of `npm config list`:

    ; "npm config ls -l" to show all defaults.

PR-URL: #16768
Credit: @happylynx
Reviewed-By: @zkat
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants