Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

fix commands using outdated command, like `npm update` #2636

Closed
wants to merge 1 commit into from

2 participants

@donpark

No description provided.

@donpark

@isaacs this bug basically no-ops npm update, meaning users who installed npm@1.1.42+ will now have to reinstall npm or node to recover. so, you might to deploy this ASAP to reduce affected users.

@isaacs
Owner

Pulled in and published on 1.1.44. Sorry, didn't see this before doing the node 0.8.3 release, but it'll be in the next one.

@isaacs isaacs closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jul 19, 2012
  1. @donpark
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  lib/outdated.js
View
2  lib/outdated.js
@@ -30,7 +30,7 @@ function outdated (args, silent, cb) {
if (typeof cb !== "function") cb = silent, silent = false
var dir = path.resolve(npm.dir, "..")
outdated_(args, dir, {}, function (er, list) {
- if (er || silent) return cb(er)
+ if (er || silent) return cb(er, list)
var outList = list.map(makePretty)
console.log(outList.join("\n"))
cb(null, list)
Something went wrong with that request. Please try again.