eslint 0.22.0 → 0.24.1

package/lib/rules/no-use-before-define.js

@@ -1,105 +1,105 @@
-/**
- * @fileoverview Rule to flag use of variables before they are defined
- * @author Ilya Volodin
- * @copyright 2013 Ilya Volodin. All rights reserved.
- */
-
-"use strict";
-
-//------------------------------------------------------------------------------
-// Constants
-//------------------------------------------------------------------------------
-
-var NO_FUNC = "nofunc";
-
-//------------------------------------------------------------------------------
-// Rule Definition
-//------------------------------------------------------------------------------
-
-module.exports = function(context) {
-
-    /**
-     * Finds variable declarations in a given scope.
-     * @param {string} name The variable name to find.
-     * @param {Scope} scope The scope to search in.
-     * @returns {Object} The variable declaration object.
-     * @private
-     */
-    function findDeclaration(name, scope) {
-        // try searching in the current scope first
-        for (var i = 0, l = scope.variables.length; i < l; i++) {
-            if (scope.variables[i].name === name) {
-                return scope.variables[i];
-            }
-        }
-        // check if there's upper scope and call recursivly till we find the variable
-        if (scope.upper) {
-            return findDeclaration(name, scope.upper);
-        }
-    }
-
-    /**
-     * Finds and validates all variables in a given scope.
-     * @param {Scope} scope The scope object.
-     * @returns {void}
-     * @private
-     */
-    function findVariablesInScope(scope) {
-        var typeOption = context.options[0];
-
-        function checkLocationAndReport(reference, declaration) {
-            if (typeOption !== NO_FUNC || declaration.defs[0].type !== "FunctionName") {
-                if (declaration.identifiers[0].range[1] > reference.identifier.range[1]) {
-                    context.report(reference.identifier, "{{a}} was used before it was defined", {a: reference.identifier.name});
-                }
-            }
-        }
-
-        scope.references.forEach(function(reference) {
-            // if the reference is resolved check for declaration location
-            // if not, it could be function invocation, try to find manually
-            if (reference.resolved && reference.resolved.identifiers.length > 0) {
-                checkLocationAndReport(reference, reference.resolved);
-            } else {
-                var declaration = findDeclaration(reference.identifier.name, scope);
-                // if there're no identifiers, this is a global environment variable
-                if (declaration && declaration.identifiers.length !== 0) {
-                    checkLocationAndReport(reference, declaration);
-                }
-            }
-        });
-    }
-
-
-    /**
-     * Validates variables inside of a node's scope.
-     * @param {ASTNode} node The node to check.
-     * @returns {void}
-     * @private
-     */
-    function findVariables() {
-        var scope = context.getScope();
-        findVariablesInScope(scope);
-    }
-
-    return {
-        "Program": function() {
-            var scope = context.getScope();
-            findVariablesInScope(scope);
-
-            // both Node.js and Modules have an extra scope
-            if (context.ecmaFeatures.globalReturn || context.ecmaFeatures.modules) {
-                findVariablesInScope(scope.childScopes[0]);
-            }
-        },
-        "FunctionExpression": findVariables,
-        "FunctionDeclaration": findVariables,
-        "ArrowFunctionExpression": findVariables
-    };
-};
-
-module.exports.schema = [
-    {
-        "enum": ["nofunc"]
-    }
-];
+/**
+ * @fileoverview Rule to flag use of variables before they are defined
+ * @author Ilya Volodin
+ * @copyright 2013 Ilya Volodin. All rights reserved.
+ */
+
+"use strict";
+
+//------------------------------------------------------------------------------
+// Constants
+//------------------------------------------------------------------------------
+
+var NO_FUNC = "nofunc";
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+module.exports = function(context) {
+
+    /**
+     * Finds variable declarations in a given scope.
+     * @param {string} name The variable name to find.
+     * @param {Scope} scope The scope to search in.
+     * @returns {Object} The variable declaration object.
+     * @private
+     */
+    function findDeclaration(name, scope) {
+        // try searching in the current scope first
+        for (var i = 0, l = scope.variables.length; i < l; i++) {
+            if (scope.variables[i].name === name) {
+                return scope.variables[i];
+            }
+        }
+        // check if there's upper scope and call recursivly till we find the variable
+        if (scope.upper) {
+            return findDeclaration(name, scope.upper);
+        }
+    }
+
+    /**
+     * Finds and validates all variables in a given scope.
+     * @param {Scope} scope The scope object.
+     * @returns {void}
+     * @private
+     */
+    function findVariablesInScope(scope) {
+        var typeOption = context.options[0];
+
+        function checkLocationAndReport(reference, declaration) {
+            if (typeOption !== NO_FUNC || declaration.defs[0].type !== "FunctionName") {
+                if (declaration.identifiers[0].range[1] > reference.identifier.range[1]) {
+                    context.report(reference.identifier, "{{a}} was used before it was defined", {a: reference.identifier.name});
+                }
+            }
+        }
+
+        scope.references.forEach(function(reference) {
+            // if the reference is resolved check for declaration location
+            // if not, it could be function invocation, try to find manually
+            if (reference.resolved && reference.resolved.identifiers.length > 0) {
+                checkLocationAndReport(reference, reference.resolved);
+            } else {
+                var declaration = findDeclaration(reference.identifier.name, scope);
+                // if there're no identifiers, this is a global environment variable
+                if (declaration && declaration.identifiers.length !== 0) {
+                    checkLocationAndReport(reference, declaration);
+                }
+            }
+        });
+    }
+
+
+    /**
+     * Validates variables inside of a node's scope.
+     * @param {ASTNode} node The node to check.
+     * @returns {void}
+     * @private
+     */
+    function findVariables() {
+        var scope = context.getScope();
+        findVariablesInScope(scope);
+    }
+
+    return {
+        "Program": function() {
+            var scope = context.getScope();
+            findVariablesInScope(scope);
+
+            // both Node.js and Modules have an extra scope
+            if (context.ecmaFeatures.globalReturn || context.ecmaFeatures.modules) {
+                findVariablesInScope(scope.childScopes[0]);
+            }
+        },
+        "FunctionExpression": findVariables,
+        "FunctionDeclaration": findVariables,
+        "ArrowFunctionExpression": findVariables
+    };
+};
+
+module.exports.schema = [
+    {
+        "enum": ["nofunc"]
+    }
+];

package/lib/rules/no-var.js

@@ -1,26 +1,26 @@
-/**
- * @fileoverview Rule to check for the usage of var.
- * @author Jamund Ferguson
- * @copyright 2014 Jamund Ferguson. All rights reserved.
- */
-
-"use strict";
-
-//------------------------------------------------------------------------------
-// Rule Definition
-//------------------------------------------------------------------------------
-
-module.exports = function(context) {
-
-    return {
-        "VariableDeclaration": function (node) {
-            if (node.kind === "var") {
-                context.report(node, "Unexpected var, use let or const instead.");
-            }
-        }
-
-    };
-
-};
-
-module.exports.schema = [];
+/**
+ * @fileoverview Rule to check for the usage of var.
+ * @author Jamund Ferguson
+ * @copyright 2014 Jamund Ferguson. All rights reserved.
+ */
+
+"use strict";
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+module.exports = function(context) {
+
+    return {
+        "VariableDeclaration": function (node) {
+            if (node.kind === "var") {
+                context.report(node, "Unexpected var, use let or const instead.");
+            }
+        }
+
+    };
+
+};
+
+module.exports.schema = [];

package/lib/rules/no-void.js

@@ -1,28 +1,28 @@
-/**
- * @fileoverview Rule to disallow use of void operator.
- * @author Mike Sidorov
- * @copyright 2014 Mike Sidorov. All rights reserved.
- */
-"use strict";
-
-//------------------------------------------------------------------------------
-// Rule Definition
-//------------------------------------------------------------------------------
-
-module.exports = function(context) {
-
-    //--------------------------------------------------------------------------
-    // Public
-    //--------------------------------------------------------------------------
-
-    return {
-        "UnaryExpression": function(node) {
-            if (node.operator === "void") {
-                context.report(node, "Expected 'undefined' and instead saw 'void'.");
-            }
-        }
-    };
-
-};
-
-module.exports.schema = [];
+/**
+ * @fileoverview Rule to disallow use of void operator.
+ * @author Mike Sidorov
+ * @copyright 2014 Mike Sidorov. All rights reserved.
+ */
+"use strict";
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+module.exports = function(context) {
+
+    //--------------------------------------------------------------------------
+    // Public
+    //--------------------------------------------------------------------------
+
+    return {
+        "UnaryExpression": function(node) {
+            if (node.operator === "void") {
+                context.report(node, "Expected 'undefined' and instead saw 'void'.");
+            }
+        }
+    };
+
+};
+
+module.exports.schema = [];

package/lib/rules/no-warning-comments.js

@@ -1,102 +1,102 @@
-/**
- * @fileoverview Rule that warns about used warning comments
- * @author Alexander Schmidt <https://github.com/lxanders>
- */
-
-"use strict";
-
-//------------------------------------------------------------------------------
-// Rule Definition
-//------------------------------------------------------------------------------
-
-module.exports = function (context) {
-
-    var configuration = context.options[0] || {},
-        warningTerms = configuration.terms || ["todo", "fixme", "xxx"],
-        location = configuration.location || "start",
-        warningRegExps;
-
-    /**
-     * Convert a warning term into a RegExp which will match a comment containing that whole word in the specified
-     * location ("start" or "anywhere"). If the term starts or ends with non word characters, then the match will not
-     * require word boundaries on that side.
-     *
-     * @param {String} term A term to convert to a RegExp
-     * @returns {RegExp} The term converted to a RegExp
-     */
-    function convertToRegExp(term) {
-        var escaped = term.replace(/[-\/\\$\^*+?.()|\[\]{}]/g, "\\$&"),
-            // If the term ends in a word character (a-z0-9_), ensure a word boundary at the end, so that substrings do
-            // not get falsely matched. eg "todo" in a string such as "mastodon".
-            // If the term ends in a non-word character, then \b won't match on the boundary to the next non-word
-            // character, which would likely be a space. For example `/\bFIX!\b/.test('FIX! blah') === false`.
-            // In these cases, use no bounding match. Same applies for the prefix, handled below.
-            suffix = /\w$/.test(term) ? "\\b" : "",
-            prefix;
-
-        if (location === "start") {
-            // When matching at the start, ignore leading whitespace, and there's no need to worry about word boundaries
-            prefix = "^\\s*";
-        } else if (/^\w/.test(term)) {
-            prefix = "\\b";
-        } else {
-            prefix = "";
-        }
-
-        return new RegExp(prefix + escaped + suffix, "i");
-    }
-
-    /**
-     * Checks the specified comment for matches of the configured warning terms and returns the matches.
-     * @param {String} comment The comment which is checked.
-     * @returns {Array} All matched warning terms for this comment.
-     */
-    function commentContainsWarningTerm(comment) {
-        var matches = [];
-
-        warningRegExps.forEach(function (regex, index) {
-            if (regex.test(comment)) {
-                matches.push(warningTerms[index]);
-            }
-        });
-
-        return matches;
-    }
-
-    /**
-     * Checks the specified node for matching warning comments and reports them.
-     * @param {ASTNode} node The AST node being checked.
-     * @returns {void} undefined.
-     */
-    function checkComment(node) {
-        var matches = commentContainsWarningTerm(node.value);
-
-        matches.forEach(function (matchedTerm) {
-            context.report(node, "Unexpected " + matchedTerm + " comment.");
-        });
-    }
-
-    warningRegExps = warningTerms.map(convertToRegExp);
-    return {
-        "BlockComment": checkComment,
-        "LineComment": checkComment
-    };
-};
-
-module.exports.schema = [
-    {
-        "type": "object",
-        "properties": {
-            "terms": {
-                "type": "array",
-                "items": {
-                    "type": "string"
-                }
-            },
-            "location": {
-                "enum": ["start", "anywhere"]
-            }
-        },
-        "additionalProperties": false
-    }
-];
+/**
+ * @fileoverview Rule that warns about used warning comments
+ * @author Alexander Schmidt <https://github.com/lxanders>
+ */
+
+"use strict";
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+module.exports = function (context) {
+
+    var configuration = context.options[0] || {},
+        warningTerms = configuration.terms || ["todo", "fixme", "xxx"],
+        location = configuration.location || "start",
+        warningRegExps;
+
+    /**
+     * Convert a warning term into a RegExp which will match a comment containing that whole word in the specified
+     * location ("start" or "anywhere"). If the term starts or ends with non word characters, then the match will not
+     * require word boundaries on that side.
+     *
+     * @param {String} term A term to convert to a RegExp
+     * @returns {RegExp} The term converted to a RegExp
+     */
+    function convertToRegExp(term) {
+        var escaped = term.replace(/[-\/\\$\^*+?.()|\[\]{}]/g, "\\$&"),
+            // If the term ends in a word character (a-z0-9_), ensure a word boundary at the end, so that substrings do
+            // not get falsely matched. eg "todo" in a string such as "mastodon".
+            // If the term ends in a non-word character, then \b won't match on the boundary to the next non-word
+            // character, which would likely be a space. For example `/\bFIX!\b/.test('FIX! blah') === false`.
+            // In these cases, use no bounding match. Same applies for the prefix, handled below.
+            suffix = /\w$/.test(term) ? "\\b" : "",
+            prefix;
+
+        if (location === "start") {
+            // When matching at the start, ignore leading whitespace, and there's no need to worry about word boundaries
+            prefix = "^\\s*";
+        } else if (/^\w/.test(term)) {
+            prefix = "\\b";
+        } else {
+            prefix = "";
+        }
+
+        return new RegExp(prefix + escaped + suffix, "i");
+    }
+
+    /**
+     * Checks the specified comment for matches of the configured warning terms and returns the matches.
+     * @param {String} comment The comment which is checked.
+     * @returns {Array} All matched warning terms for this comment.
+     */
+    function commentContainsWarningTerm(comment) {
+        var matches = [];
+
+        warningRegExps.forEach(function (regex, index) {
+            if (regex.test(comment)) {
+                matches.push(warningTerms[index]);
+            }
+        });
+
+        return matches;
+    }
+
+    /**
+     * Checks the specified node for matching warning comments and reports them.
+     * @param {ASTNode} node The AST node being checked.
+     * @returns {void} undefined.
+     */
+    function checkComment(node) {
+        var matches = commentContainsWarningTerm(node.value);
+
+        matches.forEach(function (matchedTerm) {
+            context.report(node, "Unexpected " + matchedTerm + " comment.");
+        });
+    }
+
+    warningRegExps = warningTerms.map(convertToRegExp);
+    return {
+        "BlockComment": checkComment,
+        "LineComment": checkComment
+    };
+};
+
+module.exports.schema = [
+    {
+        "type": "object",
+        "properties": {
+            "terms": {
+                "type": "array",
+                "items": {
+                    "type": "string"
+                }
+            },
+            "location": {
+                "enum": ["start", "anywhere"]
+            }
+        },
+        "additionalProperties": false
+    }
+];

package/lib/rules/no-with.js

@@ -1,22 +1,22 @@
-/**
- * @fileoverview Rule to flag use of with statement
- * @author Nicholas C. Zakas
- */
-
-"use strict";
-
-//------------------------------------------------------------------------------
-// Rule Definition
-//------------------------------------------------------------------------------
-
-module.exports = function(context) {
-
-    return {
-        "WithStatement": function(node) {
-            context.report(node, "Unexpected use of 'with' statement.");
-        }
-    };
-
-};
-
-module.exports.schema = [];
+/**
+ * @fileoverview Rule to flag use of with statement
+ * @author Nicholas C. Zakas
+ */
+
+"use strict";
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+module.exports = function(context) {
+
+    return {
+        "WithStatement": function(node) {
+            context.report(node, "Unexpected use of 'with' statement.");
+        }
+    };
+
+};
+
+module.exports.schema = [];