2351 CLI sets exit-code 1 if the command supplied was not parseable (#…

…2358) * CLI Sets exit code to 1 if the command wasn't parseable * Fix color output for errors in the cli
knex · Jan 18, 2018 · 15706c0 · 15706c0
1 parent 9f8d2ed
commit 15706c0
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/bin/cli.js b/bin/cli.js
@@ -13,9 +13,9 @@ var cliPkg = require('../package');
 
 function exit(text) {
   if (text instanceof Error) {
-    chalk.red(console.error(text.stack));
+    console.error(chalk.red(text.stack));
   } else {
-    chalk.red(console.error(text));
+    console.error(chalk.red(text));
   }
   process.exit(1);
 }
@@ -186,7 +186,8 @@ function invoke(env) {
   commander.parse(process.argv);
 
   Promise.resolve(pending).then(function() {
-    commander.help();
+    commander.outputHelp();
+    exit('Unknown command-line options, exiting');
   });
 }