tools: add -F flag for fixing lint issues

Add `-F` flag to `jslint.js` which enables the automatic fixing of issues that are fixable. PR-URL: #6483 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Brian White <[email protected]>
nodejs · May 4, 2016 · 08e0884 · 08e0884
1 parent 7c30f15
commit 08e0884
Showing 1 changed file with 13 additions and 2 deletions.
diff --git a/tools/jslint.js b/tools/jslint.js
@@ -13,9 +13,15 @@ const CLIEngine = require('./eslint').CLIEngine;
 const glob = require('./eslint/node_modules/glob');
 
 const cwd = process.cwd();
-const cli = new CLIEngine({
+const cliOptions = {
   rulePaths: rulesDirs
-});
+};
+
+// Check if we should fix errors that are fixable
+if (process.argv.indexOf('-F') !== -1)
+  cliOptions.fix = true;
+
+const cli = new CLIEngine(cliOptions);
 
 if (cluster.isMaster) {
   var numCPUs = 1;
@@ -240,6 +246,11 @@ if (cluster.isMaster) {
     if (files instanceof Array) {
       // Lint some files
       const report = cli.executeOnFiles(files);
+
+      // If we were asked to fix the fixable issues, do so.
+      if (cliOptions.fix)
+        CLIEngine.outputFixes(report);
+
       if (config.sendAll) {
         // Return both success and error results