eslint 0.22.0 → 0.24.1

package/lib/file-finder.js

@@ -1,167 +1,167 @@
-/**
- * @fileoverview Util class to find config files.
- * @author Aliaksei Shytkin
- * @copyright 2014 Michael McLaughlin. All rights reserved.
- */
-"use strict";
-
-//------------------------------------------------------------------------------
-// Requirements
-//------------------------------------------------------------------------------
-
-var fs = require("fs"),
-    path = require("path");
-
-//------------------------------------------------------------------------------
-// Helpers
-//------------------------------------------------------------------------------
-
-/**
- * Get the entries for a directory. Including a try-catch may be detrimental to
- * function performance, so move it out here a separate function.
- * @param {string} directory The directory to search in.
- * @returns {string[]} The entries in the directory or an empty array on error.
- * @private
- */
-function getDirectoryEntries(directory) {
-    try {
-        return fs.readdirSync(directory);
-    } catch (ex) {
-        return [];
-    }
-}
-
-//------------------------------------------------------------------------------
-// API
-//------------------------------------------------------------------------------
-
-/**
- * FileFinder
- * @constructor
- * @param {...string} arguments The basename(s) of the file(s) to find.
- */
-function FileFinder() {
-    this.fileNames = Array.prototype.slice.call(arguments);
-    this.cache = {};
-}
-
-/**
- * Find one instance of a specified file name in directory or in a parent directory.
- * Cache the results.
- * Does not check if a matching directory entry is a file, and intentionally
- * only searches for the first file name in this.fileNames.
- * Is currently used by lib/ignored_paths.js to find an .eslintignore file.
- * @param  {string} directory The directory to start the search from.
- * @returns {string} Path of the file found, or an empty string if not found.
- */
-FileFinder.prototype.findInDirectoryOrParents = function (directory) {
-    var cache = this.cache,
-        child,
-        dirs,
-        filePath,
-        i,
-        name,
-        searched;
-
-    if (!directory) {
-        directory = process.cwd();
-    }
-
-    if (cache.hasOwnProperty(directory)) {
-        return cache[directory];
-    }
-
-    dirs = [];
-    searched = 0;
-    name = this.fileNames[0];
-
-    while (directory !== child) {
-        dirs[searched++] = directory;
-
-        if (getDirectoryEntries(directory).indexOf(name) !== -1) {
-            filePath = path.resolve(directory, name);
-            break;
-        }
-
-        child = directory;
-
-        // Assign parent directory to directory.
-        directory = path.dirname(directory);
-    }
-
-    for (i = 0; i < searched; i++) {
-        cache[dirs[i]] = filePath;
-    }
-
-    return filePath || String();
-};
-
-/**
- * Find all instances of files with the specified file names, in directory and
- * parent directories. Cache the results.
- * Does not check if a matching directory entry is a file.
- * Searches for all the file names in this.fileNames.
- * Is currently used by lib/config.js to find .eslintrc and package.json files.
- * @param  {string} directory The directory to start the search from.
- * @returns {string[]} The file paths found.
- */
-FileFinder.prototype.findAllInDirectoryAndParents = function (directory) {
-    var cache = this.cache,
-        child,
-        dirs,
-        name,
-        fileNames,
-        fileNamesCount,
-        filePath,
-        i,
-        j,
-        searched;
-
-    if (!directory) {
-        directory = process.cwd();
-    }
-
-    if (cache.hasOwnProperty(directory)) {
-        return cache[directory];
-    }
-
-    dirs = [];
-    searched = 0;
-    fileNames = this.fileNames;
-    fileNamesCount = fileNames.length;
-
-    do {
-        dirs[searched++] = directory;
-        cache[directory] = [];
-
-        for (i = 0; i < fileNamesCount; i++) {
-            name = fileNames[i];
-
-            if (getDirectoryEntries(directory).indexOf(name) !== -1) {
-                filePath = path.resolve(directory, name);
-
-                // Add the file path to the cache of each directory searched.
-                for (j = 0; j < searched; j++) {
-                    cache[dirs[j]].push(filePath);
-                }
-            }
-        }
-        child = directory;
-
-        // Assign parent directory to directory.
-        directory = path.dirname(directory);
-
-        if (directory === child) {
-            return cache[dirs[0]];
-        }
-    } while (!cache.hasOwnProperty(directory));
-
-    // Add what has been cached previously to the cache of each directory searched.
-    for (i = 0; i < searched; i++) {
-        dirs.push.apply(cache[dirs[i]], cache[directory]);
-    }
-
-    return cache[dirs[0]];
-};
-
-module.exports = FileFinder;
+/**
+ * @fileoverview Util class to find config files.
+ * @author Aliaksei Shytkin
+ * @copyright 2014 Michael McLaughlin. All rights reserved.
+ */
+"use strict";
+
+//------------------------------------------------------------------------------
+// Requirements
+//------------------------------------------------------------------------------
+
+var fs = require("fs"),
+    path = require("path");
+
+//------------------------------------------------------------------------------
+// Helpers
+//------------------------------------------------------------------------------
+
+/**
+ * Get the entries for a directory. Including a try-catch may be detrimental to
+ * function performance, so move it out here a separate function.
+ * @param {string} directory The directory to search in.
+ * @returns {string[]} The entries in the directory or an empty array on error.
+ * @private
+ */
+function getDirectoryEntries(directory) {
+    try {
+        return fs.readdirSync(directory);
+    } catch (ex) {
+        return [];
+    }
+}
+
+//------------------------------------------------------------------------------
+// API
+//------------------------------------------------------------------------------
+
+/**
+ * FileFinder
+ * @constructor
+ * @param {...string} arguments The basename(s) of the file(s) to find.
+ */
+function FileFinder() {
+    this.fileNames = Array.prototype.slice.call(arguments);
+    this.cache = {};
+}
+
+/**
+ * Find one instance of a specified file name in directory or in a parent directory.
+ * Cache the results.
+ * Does not check if a matching directory entry is a file, and intentionally
+ * only searches for the first file name in this.fileNames.
+ * Is currently used by lib/ignored_paths.js to find an .eslintignore file.
+ * @param  {string} directory The directory to start the search from.
+ * @returns {string} Path of the file found, or an empty string if not found.
+ */
+FileFinder.prototype.findInDirectoryOrParents = function (directory) {
+    var cache = this.cache,
+        child,
+        dirs,
+        filePath,
+        i,
+        name,
+        searched;
+
+    if (!directory) {
+        directory = process.cwd();
+    }
+
+    if (cache.hasOwnProperty(directory)) {
+        return cache[directory];
+    }
+
+    dirs = [];
+    searched = 0;
+    name = this.fileNames[0];
+
+    while (directory !== child) {
+        dirs[searched++] = directory;
+
+        if (getDirectoryEntries(directory).indexOf(name) !== -1 && fs.statSync(path.resolve(directory, name)).isFile()) {
+            filePath = path.resolve(directory, name);
+            break;
+        }
+
+        child = directory;
+
+        // Assign parent directory to directory.
+        directory = path.dirname(directory);
+    }
+
+    for (i = 0; i < searched; i++) {
+        cache[dirs[i]] = filePath;
+    }
+
+    return filePath || String();
+};
+
+/**
+ * Find all instances of files with the specified file names, in directory and
+ * parent directories. Cache the results.
+ * Does not check if a matching directory entry is a file.
+ * Searches for all the file names in this.fileNames.
+ * Is currently used by lib/config.js to find .eslintrc and package.json files.
+ * @param  {string} directory The directory to start the search from.
+ * @returns {string[]} The file paths found.
+ */
+FileFinder.prototype.findAllInDirectoryAndParents = function (directory) {
+    var cache = this.cache,
+        child,
+        dirs,
+        name,
+        fileNames,
+        fileNamesCount,
+        filePath,
+        i,
+        j,
+        searched;
+
+    if (!directory) {
+        directory = process.cwd();
+    }
+
+    if (cache.hasOwnProperty(directory)) {
+        return cache[directory];
+    }
+
+    dirs = [];
+    searched = 0;
+    fileNames = this.fileNames;
+    fileNamesCount = fileNames.length;
+
+    do {
+        dirs[searched++] = directory;
+        cache[directory] = [];
+
+        for (i = 0; i < fileNamesCount; i++) {
+            name = fileNames[i];
+
+            if (getDirectoryEntries(directory).indexOf(name) !== -1 && fs.statSync(path.resolve(directory, name)).isFile()) {
+                filePath = path.resolve(directory, name);
+
+                // Add the file path to the cache of each directory searched.
+                for (j = 0; j < searched; j++) {
+                    cache[dirs[j]].push(filePath);
+                }
+            }
+        }
+        child = directory;
+
+        // Assign parent directory to directory.
+        directory = path.dirname(directory);
+
+        if (directory === child) {
+            return cache[dirs[0]];
+        }
+    } while (!cache.hasOwnProperty(directory));
+
+    // Add what has been cached previously to the cache of each directory searched.
+    for (i = 0; i < searched; i++) {
+        dirs.push.apply(cache[dirs[i]], cache[directory]);
+    }
+
+    return cache[dirs[0]];
+};
+
+module.exports = FileFinder;

package/lib/formatters/checkstyle.js

@@ -1,68 +1,68 @@
-/**
- * @fileoverview CheckStyle XML reporter
- * @author Ian Christian Myers
- */
-"use strict";
-
-//------------------------------------------------------------------------------
-// Helper Functions
-//------------------------------------------------------------------------------
-
-function getMessageType(message) {
-    if (message.fatal || message.severity === 2) {
-        return "error";
-    } else {
-        return "warning";
-    }
-}
-
-function xmlEscape(s) {
-    return ("" + s).replace(/[<>&"']/g, function(c) {
-        switch (c) {
-            case "<":
-                return "&lt;";
-            case ">":
-                return "&gt;";
-            case "&":
-                return "&amp;";
-            case "\"":
-                return "&quot;";
-            case "'":
-                return "&apos;";
-            // no default
-        }
-    });
-}
-
-//------------------------------------------------------------------------------
-// Public Interface
-//------------------------------------------------------------------------------
-
-module.exports = function(results) {
-
-    var output = "";
-
-    output += "<?xml version=\"1.0\" encoding=\"utf-8\"?>";
-    output += "<checkstyle version=\"4.3\">";
-
-    results.forEach(function(result) {
-        var messages = result.messages;
-
-        output += "<file name=\"" + xmlEscape(result.filePath) + "\">";
-
-        messages.forEach(function(message) {
-            output += "<error line=\"" + xmlEscape(message.line) + "\" " +
-                "column=\"" + xmlEscape(message.column) + "\" " +
-                "severity=\"" + xmlEscape(getMessageType(message)) + "\" " +
-                "message=\"" + xmlEscape(message.message) +
-                (message.ruleId ? " (" + message.ruleId + ")" : "") + "\" />";
-        });
-
-        output += "</file>";
-
-    });
-
-    output += "</checkstyle>";
-
-    return output;
-};
+/**
+ * @fileoverview CheckStyle XML reporter
+ * @author Ian Christian Myers
+ */
+"use strict";
+
+//------------------------------------------------------------------------------
+// Helper Functions
+//------------------------------------------------------------------------------
+
+function getMessageType(message) {
+    if (message.fatal || message.severity === 2) {
+        return "error";
+    } else {
+        return "warning";
+    }
+}
+
+function xmlEscape(s) {
+    return ("" + s).replace(/[<>&"']/g, function(c) {
+        switch (c) {
+            case "<":
+                return "&lt;";
+            case ">":
+                return "&gt;";
+            case "&":
+                return "&amp;";
+            case "\"":
+                return "&quot;";
+            case "'":
+                return "&apos;";
+            // no default
+        }
+    });
+}
+
+//------------------------------------------------------------------------------
+// Public Interface
+//------------------------------------------------------------------------------
+
+module.exports = function(results) {
+
+    var output = "";
+
+    output += "<?xml version=\"1.0\" encoding=\"utf-8\"?>";
+    output += "<checkstyle version=\"4.3\">";
+
+    results.forEach(function(result) {
+        var messages = result.messages;
+
+        output += "<file name=\"" + xmlEscape(result.filePath) + "\">";
+
+        messages.forEach(function(message) {
+            output += "<error line=\"" + xmlEscape(message.line) + "\" " +
+                "column=\"" + xmlEscape(message.column) + "\" " +
+                "severity=\"" + xmlEscape(getMessageType(message)) + "\" " +
+                "message=\"" + xmlEscape(message.message) +
+                (message.ruleId ? " (" + message.ruleId + ")" : "") + "\" />";
+        });
+
+        output += "</file>";
+
+    });
+
+    output += "</checkstyle>";
+
+    return output;
+};

package/lib/formatters/compact.js

@@ -1,53 +1,53 @@
-/**
- * @fileoverview Compact reporter
- * @author Nicholas C. Zakas
- */
-"use strict";
-
-//------------------------------------------------------------------------------
-// Helper Functions
-//------------------------------------------------------------------------------
-
-function getMessageType(message) {
-    if (message.fatal || message.severity === 2) {
-        return "Error";
-    } else {
-        return "Warning";
-    }
-}
-
-
-//------------------------------------------------------------------------------
-// Public Interface
-//------------------------------------------------------------------------------
-
-module.exports = function(results) {
-
-    var output = "",
-        total = 0;
-
-    results.forEach(function(result) {
-
-        var messages = result.messages;
-        total += messages.length;
-
-        messages.forEach(function(message) {
-
-            output += result.filePath + ": ";
-            output += "line " + (message.line || 0);
-            output += ", col " + (message.column || 0);
-            output += ", " + getMessageType(message);
-            output += " - " + message.message;
-            output += message.ruleId ? " (" + message.ruleId + ")" : "";
-            output += "\n";
-
-        });
-
-    });
-
-    if (total > 0) {
-        output += "\n" + total + " problem" + (total !== 1 ? "s" : "");
-    }
-
-    return output;
-};
+/**
+ * @fileoverview Compact reporter
+ * @author Nicholas C. Zakas
+ */
+"use strict";
+
+//------------------------------------------------------------------------------
+// Helper Functions
+//------------------------------------------------------------------------------
+
+function getMessageType(message) {
+    if (message.fatal || message.severity === 2) {
+        return "Error";
+    } else {
+        return "Warning";
+    }
+}
+
+
+//------------------------------------------------------------------------------
+// Public Interface
+//------------------------------------------------------------------------------
+
+module.exports = function(results) {
+
+    var output = "",
+        total = 0;
+
+    results.forEach(function(result) {
+
+        var messages = result.messages;
+        total += messages.length;
+
+        messages.forEach(function(message) {
+
+            output += result.filePath + ": ";
+            output += "line " + (message.line || 0);
+            output += ", col " + (message.column || 0);
+            output += ", " + getMessageType(message);
+            output += " - " + message.message;
+            output += message.ruleId ? " (" + message.ruleId + ")" : "";
+            output += "\n";
+
+        });
+
+    });
+
+    if (total > 0) {
+        output += "\n" + total + " problem" + (total !== 1 ? "s" : "");
+    }
+
+    return output;
+};

package/lib/formatters/jslint-xml.js

@@ -1,40 +1,40 @@
-/**
- * @fileoverview JSLint XML reporter
- * @author Ian Christian Myers
- */
-"use strict";
-
-var xmlescape = require("xml-escape");
-
-//------------------------------------------------------------------------------
-// Public Interface
-//------------------------------------------------------------------------------
-
-module.exports = function(results) {
-
-    var output = "";
-
-    output += "<?xml version=\"1.0\" encoding=\"utf-8\"?>";
-    output += "<jslint>";
-
-    results.forEach(function(result) {
-        var messages = result.messages;
-
-        output += "<file name=\"" + result.filePath + "\">";
-
-        messages.forEach(function(message) {
-            output += "<issue line=\"" + message.line + "\" " +
-                "char=\"" + message.column + "\" " +
-                "evidence=\"" + xmlescape(message.source || "") + "\" " +
-                "reason=\"" + xmlescape(message.message || "") +
-                (message.ruleId ? " (" + message.ruleId + ")" : "") + "\" />";
-        });
-
-        output += "</file>";
-
-    });
-
-    output += "</jslint>";
-
-    return output;
-};
+/**
+ * @fileoverview JSLint XML reporter
+ * @author Ian Christian Myers
+ */
+"use strict";
+
+var xmlescape = require("xml-escape");
+
+//------------------------------------------------------------------------------
+// Public Interface
+//------------------------------------------------------------------------------
+
+module.exports = function(results) {
+
+    var output = "";
+
+    output += "<?xml version=\"1.0\" encoding=\"utf-8\"?>";
+    output += "<jslint>";
+
+    results.forEach(function(result) {
+        var messages = result.messages;
+
+        output += "<file name=\"" + result.filePath + "\">";
+
+        messages.forEach(function(message) {
+            output += "<issue line=\"" + message.line + "\" " +
+                "char=\"" + message.column + "\" " +
+                "evidence=\"" + xmlescape(message.source || "") + "\" " +
+                "reason=\"" + xmlescape(message.message || "") +
+                (message.ruleId ? " (" + message.ruleId + ")" : "") + "\" />";
+        });
+
+        output += "</file>";
+
+    });
+
+    output += "</jslint>";
+
+    return output;
+};