[[FEAT]] add varstmt enforcement option to disallow use of Variable…

…Statements

Closes #1549

src/jshint.js

@@ -178,7 +178,8 @@ var JSHINT = (function() {
   function processenforceall() {
     if (state.option.enforceall) {
       for (var enforceopt in options.bool.enforcing) {
-        if (state.option[enforceopt] === undefined) {
+        if (state.option[enforceopt] === undefined &&
+            !options.noenforceall[enforceopt]) {
           state.option[enforceopt] = true;
         }
       }
@@ -3507,6 +3508,10 @@ var JSHINT = (function() {
         break;
       }
 
+      if (state.option.varstmt) {
+        warning("W132", this);
+      }
+
       this.first = this.first.concat(names);
 
       if (state.tokens.next.id === "=") {

src/messages.js

@@ -206,7 +206,8 @@ var warnings = {
   W129: "'{a}' is defined in a future version of JavaScript. Use a " +
     "different variable name to avoid migration issues.",
   W130: "Invalid element after rest element.",
-  W131: "Invalid parameter after rest parameter."
+  W131: "Invalid parameter after rest parameter.",
+  W132: "`var` declarations are forbidden. Use `let` or `const` instead."
 };
 
 var info = {

src/options.js

@@ -267,6 +267,16 @@ exports.bool = {
      */
     singleGroups: false,
 
+    /**
+     * When set to true, the use of VariableStatements are forbidden.
+     * For example:
+     *
+     *     // jshint varstmt: true
+     *
+     *     var a; // Warning: `var` declarations are forbidden. Use `let` or `const` instead.
+     */
+    varstmt: false,
+
     /**
      * This option is a short hand for the most strict JSHint configuration. It
      * enables all enforcing options and disables all relaxing options.
@@ -945,3 +955,9 @@ exports.removed = {
   smarttabs: true,
   trailing: true
 };
+
+// Add options here which should not be automatically enforced by
+// `enforceall`.
+exports.noenforceall = {
+  varstmt: true
+};

tests/unit/options.js

@@ -2290,3 +2290,25 @@ exports.futureHostile = function (test) {
 
   test.done();
 };
+
+
+exports.varstmt = function (test) {
+  var code = [
+    "var x;",
+    "var y = 5;",
+    "var fn = function() {",
+    "  var x;",
+    "  var y = 5;",
+    "};"
+  ];
+
+  TestRun(test)
+    .addError(1, "`var` declarations are forbidden. Use `let` or `const` instead.")
+    .addError(2, "`var` declarations are forbidden. Use `let` or `const` instead.")
+    .addError(3, "`var` declarations are forbidden. Use `let` or `const` instead.")
+    .addError(4, "`var` declarations are forbidden. Use `let` or `const` instead.")
+    .addError(5, "`var` declarations are forbidden. Use `let` or `const` instead.")
+    .test(code, { varstmt: true });
+
+  test.done();
+};