remove update arg again, it's covered by --include

This commit is contained in:
silverwind 2018-10-04 07:58:20 +02:00
parent 1b5cf98c1d
commit 947fd6f7fb
Signed by: silverwind
GPG Key ID: 2E62B41C93869443
2 changed files with 5 additions and 7 deletions

@ -19,7 +19,7 @@ $ npm i -g updates
usage: updates [options]
Options:
-u, --update [<pkg,...>] Update packages and write package.json
-u, --update Update packages and write package.json
-p, --prerelease [<pkg,...>] Consider prerelease versions
-g, --greatest [<pkg,...>] Prefer greatest over latest version
-i, --include <pkg,...> Only include given packages

@ -9,6 +9,7 @@ const args = require("minimist")(process.argv.slice(2), {
"h", "help",
"j", "json",
"n", "no-color",
"u", "update",
"v", "version",
],
string: [
@ -16,7 +17,6 @@ const args = require("minimist")(process.argv.slice(2), {
"g", "greatest",
"p", "prerelease",
"r", "registry",
"u", "update",
],
default: {
"registry": "https://registry.npmjs.org/",
@ -41,7 +41,7 @@ if (args.help) {
process.stdout.write(`usage: updates [options]
Options:
-u, --update [<pkg,...>] Update packages and write package.json
-u, --update Update packages and write package.json
-p, --prerelease [<pkg,...>] Consider prerelease versions
-g, --greatest [<pkg,...>] Prefer greatest over latest version
-i, --include <pkg,...> Only include given packages
@ -74,7 +74,6 @@ if (args["no-color"]) process.env.FORCE_COLOR = "0";
const greatest = parseMixedArg(args.greatest);
const prerelease = parseMixedArg(args.prerelease);
const update = parseMixedArg(args.update);
const registry = args.registry.endsWith("/") ? args.registry : args.registry + "/";
const packageFile = args.file || require("find-up").sync("package.json");
@ -111,7 +110,6 @@ if (args.exclude) exclude = args.exclude.split(",");
for (const key of dependencyTypes) {
if (pkg[key]) {
const names = Object.keys(pkg[key])
.filter(name => Array.isArray(update) ? update.includes(name) : true)
.filter(name => !include ? true : include.includes(name))
.filter(name => !exclude ? true : !exclude.includes(name));
@ -125,7 +123,7 @@ for (const key of dependencyTypes) {
}
if (!Object.keys(deps).length) {
if (Array.isArray(update) || include || exclude) {
if (include || exclude) {
finish(new Error("No packages match the given filters"));
} else {
finish(new Error("No packages found"));
@ -167,7 +165,7 @@ Promise.all(Object.keys(deps).map(name => fetch(buildUrl(name)).then(r => r.json
finish("All packages are up to date.");
}
if (args.update === false) {
if (!args.update) {
finish();
}