f601d040b5
Update ESLint to 2.1.0. ESLint has a number of potentially-useful new features but this change attempts to be minimal in its changes. However, some things could not be avoided reasonably. ESLint 2.1.0 found a few lint issues that ESLing 1.x missed with template strings that did not take advantage of any features of template strings, and `let` declarations where `const` sufficed. Additionally, ESLint 2.1.0 removes some granularity around enabling ES6 features. Some features (e.g., spread operator) that had been turned off in our configuration for ESLint 1.x are now permitted. PR-URL: https://github.com/nodejs/node/pull/5214 Reviewed-By: Michaël Zasso <mic.besace@gmail.com> Reviewed-By: jbergstroem - Johan Bergström <bugs@bergstroem.nu> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Roman Reiss <me@silverwind.io> Reviewed-By: Myles Borins <myles.borins@gmail.com>
90 lines
2.7 KiB
JavaScript
90 lines
2.7 KiB
JavaScript
/**
|
|
* @fileoverview Stylish reporter
|
|
* @author Sindre Sorhus
|
|
*/
|
|
"use strict";
|
|
|
|
var chalk = require("chalk"),
|
|
table = require("text-table");
|
|
|
|
//------------------------------------------------------------------------------
|
|
// Helpers
|
|
//------------------------------------------------------------------------------
|
|
|
|
/**
|
|
* Given a word and a count, append an s if count is not one.
|
|
* @param {string} word A word in its singular form.
|
|
* @param {int} count A number controlling whether word should be pluralized.
|
|
* @returns {string} The original word with an s on the end if count is not one.
|
|
*/
|
|
function pluralize(word, count) {
|
|
return (count === 1 ? word : word + "s");
|
|
}
|
|
|
|
//------------------------------------------------------------------------------
|
|
// Public Interface
|
|
//------------------------------------------------------------------------------
|
|
|
|
module.exports = function(results) {
|
|
|
|
var output = "\n",
|
|
total = 0,
|
|
errors = 0,
|
|
warnings = 0,
|
|
summaryColor = "yellow";
|
|
|
|
results.forEach(function(result) {
|
|
var messages = result.messages;
|
|
|
|
if (messages.length === 0) {
|
|
return;
|
|
}
|
|
|
|
total += messages.length;
|
|
output += chalk.underline(result.filePath) + "\n";
|
|
|
|
output += table(
|
|
messages.map(function(message) {
|
|
var messageType;
|
|
|
|
if (message.fatal || message.severity === 2) {
|
|
messageType = chalk.red("error");
|
|
summaryColor = "red";
|
|
errors++;
|
|
} else {
|
|
messageType = chalk.yellow("warning");
|
|
warnings++;
|
|
}
|
|
|
|
return [
|
|
"",
|
|
message.line || 0,
|
|
message.column || 0,
|
|
messageType,
|
|
message.message.replace(/\.$/, ""),
|
|
chalk.dim(message.ruleId || "")
|
|
];
|
|
}),
|
|
{
|
|
align: ["", "r", "l"],
|
|
stringLength: function(str) {
|
|
return chalk.stripColor(str).length;
|
|
}
|
|
}
|
|
).split("\n").map(function(el) {
|
|
return el.replace(/(\d+)\s+(\d+)/, function(m, p1, p2) {
|
|
return chalk.dim(p1 + ":" + p2);
|
|
});
|
|
}).join("\n") + "\n\n";
|
|
});
|
|
|
|
if (total > 0) {
|
|
output += chalk[summaryColor].bold([
|
|
"\u2716 ", total, pluralize(" problem", total),
|
|
" (", errors, pluralize(" error", errors), ", ",
|
|
warnings, pluralize(" warning", warnings), ")\n"
|
|
].join(""));
|
|
}
|
|
|
|
return total > 0 ? output : "";
|
|
};
|