Add ping command #5788

Closed
wants to merge 1 commit into
from

Projects

None yet

4 participants

@michaelnisi
Contributor
@yorkie yorkie commented on an outdated diff Jul 24, 2014
@@ -0,0 +1,21 @@
+
+var npm = require("./npm.js")
+
+module.exports = ping
+
+ping.usage = "npm ping"
+ping.usage = "\n"
+ping.usage = "ping registry"
@yorkie
yorkie Jul 24, 2014 Contributor

how it does make sense? i guess using +=?

@yorkie yorkie commented on an outdated diff Jul 24, 2014
@@ -0,0 +1,21 @@
+
+var npm = require("./npm.js")
+
+module.exports = ping
+
+ping.usage = "npm ping"
+ping.usage = "\n"
@yorkie
yorkie Jul 24, 2014 Contributor

ditto

@michaelnisi
Contributor

Sorry

@yorkie
Contributor
yorkie commented Jul 24, 2014

Great :p

@othiym23
Contributor

We haven't forgotten about this! We just need to get it together to land all the pieces and roll it out as a unit.

@michaelnisi
Contributor

Telepathy? I'm back—let me know if I can do something (for this or anything else).

@iarna iarna added this to the 2.13.0 milestone Jun 29, 2015
@zkat zkat added a commit that referenced this pull request Jun 30, 2015
@michaelnisi @zkat michaelnisi + zkat Add ping command
Fixes #5750

PR-URL: #5788
f1f7a85
@iarna iarna added a commit that referenced this pull request Jul 1, 2015
@michaelnisi @iarna michaelnisi + iarna npm: Add ping command
Fixes #5750

PR-URL: #5788
69e4c22
@iarna iarna added a commit that referenced this pull request Jul 1, 2015
@michaelnisi @iarna michaelnisi + iarna npm: Add ping command
Fixes #5750

PR-URL: #5788
311db70
@iarna
Member
iarna commented Jul 1, 2015

Landed in 2.x as f1f7a85

@iarna
Member
iarna commented Jul 1, 2015

Landed in master as 311db70

@iarna iarna closed this Jul 1, 2015
@iarna iarna removed the review label Jul 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment