Skip to content

add a note about the details returned by the command - #111

Closed
fharper wants to merge 1 commit into
latestfrom
fharper/pingdocdetail
Closed

fharper wants to merge 1 commit into
latestfrom
fharper/pingdocdetail

Conversation

@fharper

@fharper fharper commented Dec 6, 2018

Copy link
Copy Markdown
Contributor

No description provided.

@fharper
fharper requested a review from a team as a code owner December 6, 2018 16:59
@zkat zkat added the docs label Dec 6, 2018

@zkat zkat left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Heya, thanks for the patch. The new npm-registry-fetch branch includes patches to ping that make the JSON output consistently look like this:

{
  "registry": "https://registry.npmjs.org/",
  "time": 250,
  "details": {}
}

Additionally, the default output is something like this:

➜ npm ping
npm notice PING https://registry.npmjs.org/
npm notice PONG 0.417ms

@fharper fharper closed this Dec 7, 2018
@fharper
fharper deleted the fharper/pingdocdetail branch December 7, 2018 15:18
github-actions Bot added a commit to Kevinlee7250/cli that referenced this pull request Jul 2, 2026
github-actions Bot added a commit to Kevinlee7250/cli that referenced this pull request Jul 12, 2026
github-actions Bot added a commit to Kevinlee7250/cli that referenced this pull request Aug 7, 2026
github-actions Bot added a commit to Kevinlee7250/cli that referenced this pull request Sep 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants