Only set exitCode instead of exiting immediately (palantir#2867)

pablonete · Jun 2, 2017 · d87c52d · d87c52d
1 parent 07eb8da
commit d87c52d
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/package.json b/package.json
@@ -29,7 +29,7 @@
     "lint:global": "tslint --project test/tsconfig.json --format stylish --type-check # test includes 'src' too",
     "lint:from-bin": "node bin/tslint --project test/tsconfig.json --format stylish",
     "test": "npm-run-all test:pre -p test:mocha test:rules",
-    "test:pre": "cd ./test/config && npm install",
+    "test:pre": "cd ./test/config && npm install --no-save",
     "test:mocha": "mocha --reporter spec --colors \"build/test/**/*Tests.js\"",
     "test:rules": "node ./build/test/ruleTestRunner.js",
     "verify": "npm-run-all clean compile lint test docs",

diff --git a/src/tslint-cli.ts b/src/tslint-cli.ts
@@ -254,7 +254,12 @@ run({
 }, {
     log,
     error: (m) => console.error(m),
-}).then(process.exit);
+}).then((rc) => {
+    process.exitCode = rc;
+}).catch((e) => {
+    console.error(e);
+    process.exitCode = 1;
+});
 
 function optionUsageTag({short, name}: Option) {
     return short !== undefined ? `-${short}, --${name}` : `--${name}`;