From 08e0884ae02349c3f2b8bb8cee5406c0adade944 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Fri, 29 Apr 2016 16:47:18 -0700 Subject: [PATCH] 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: https://github.com/nodejs/node/pull/6483 Reviewed-By: James M Snell Reviewed-By: Brian White --- tools/jslint.js | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/tools/jslint.js b/tools/jslint.js index 64193f15a7c343..5aa6079dea9fa5 100644 --- 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