Merge pull request #158 from Recras/set-exit-code-on-error

Set error state if there is any error
rdio · Jul 30, 2015 · a1a9ff7 · a1a9ff7
2 parents 5c51845 + fcdf2bf
commit a1a9ff7
Showing 1 changed file with 16 additions and 8 deletions.
diff --git a/lib/run.js b/lib/run.js
@@ -105,7 +105,7 @@ function handleJavascript(fullPath, original) {
       js = jsfmt.parseAST(JSON.parse(js));
     } catch ( err ) {
       console.error(relativePath, err.message);
-      return;
+      return false;
     }
   }
 
@@ -123,16 +123,17 @@ function handleJavascript(fullPath, original) {
       });
     } catch ( err ) {
       console.error(relativePath, err.message);
+      return false;
     }
-    return;
+    return true;
   }
 
   if (argv['--rewrite']) {
     try {
       js = jsfmt.rewrite(js, argv['--rewrite']).toString();
     } catch ( err ) {
       console.error(relativePath, err);
-      return;
+      return false;
     }
   }
 
@@ -145,7 +146,7 @@ function handleJavascript(fullPath, original) {
       }
     } catch ( err ) {
       console.error(relativePath, err);
-      return;
+      return false;
     }
   }
 
@@ -161,7 +162,7 @@ function handleJavascript(fullPath, original) {
         var msg = util.format('Error: %s Line: %s Column: %s', error.description, error.lineNumber, error.column);
         console.error(msg);
       });
-      process.exit(-1);
+      return false;
     }
   }
 
@@ -186,6 +187,7 @@ function handleJavascript(fullPath, original) {
     // Print to stdout
     process.stdout.write(js);
   }
+  return true;
 }
 
 function handleDirectory(currentPath, callback) {
@@ -214,11 +216,15 @@ if (paths.length > 0) {
         if (fs.statSync(fullPath).isDirectory()) {
           handleDirectory(fullPath, function(paths) {
             _.each(paths, function(fullPath) {
-              handleJavascript(path.normalize(fullPath), fs.readFileSync(fullPath, 'utf-8'));
+              if (!handleJavascript(path.normalize(fullPath), fs.readFileSync(fullPath, 'utf-8'))) {
+                process.exitCode = -1;
+              }
             });
           });
         } else {
-          handleJavascript(fullPath, fs.readFileSync(fullPath, 'utf-8'));
+          if (!handleJavascript(fullPath, fs.readFileSync(fullPath, 'utf-8'))) {
+            process.exitCode = -1;
+          }
         }
       });
     });
@@ -233,6 +239,8 @@ if (paths.length > 0) {
     }
   });
   process.stdin.on('end', function() {
-    handleJavascript('stdin', js);
+    if (!handleJavascript('stdin', js)) {
+      process.exitCode = -1;
+    }
   });
 }