Skip to content

Commit

Permalink
Build: gensite passes rulesMeta to formatter rendering (#11567)
Browse files Browse the repository at this point in the history
* Build: gensite passes rulesMeta to formatter rendering

* Chore: Switching to forEach for readability
  • Loading branch information
platinumazure authored and ilyavolodin committed Mar 30, 2019
1 parent c06d38c commit dfef227
Showing 1 changed file with 21 additions and 9 deletions.
30 changes: 21 additions & 9 deletions Makefile.js
Expand Up @@ -430,16 +430,17 @@ function getFormatterResults() {
const stripAnsi = require("strip-ansi");

const formatterFiles = fs.readdirSync("./lib/formatters/"),
rules = {
"no-else-return": "warn",
indent: ["warn", 4],
"space-unary-ops": "error",
semi: ["warn", "always"],
"consistent-return": "error"
},
cli = new CLIEngine({
useEslintrc: false,
baseConfig: { extends: "eslint:recommended" },
rules: {
"no-else-return": 1,
indent: [1, 4],
"space-unary-ops": 2,
semi: [1, "always"],
"consistent-return": 2
}
rules
}),
codeString = [
"function addOne(i) {",
Expand All @@ -450,15 +451,26 @@ function getFormatterResults() {
" }",
"};"
].join("\n"),
rawMessages = cli.executeOnText(codeString, "fullOfProblems.js", true);
rawMessages = cli.executeOnText(codeString, "fullOfProblems.js", true),
rulesMap = cli.getRules(),
rulesMeta = {};

Object.keys(rules).forEach(ruleId => {
rulesMeta[ruleId] = rulesMap.get(ruleId).meta;
});

return formatterFiles.reduce((data, filename) => {
const fileExt = path.extname(filename),
name = path.basename(filename, fileExt);

if (fileExt === ".js") {
const formattedOutput = cli.getFormatter(name)(
rawMessages.results,
{ rulesMeta }
);

data.formatterResults[name] = {
result: stripAnsi(cli.getFormatter(name)(rawMessages.results))
result: stripAnsi(formattedOutput)
};
}
return data;
Expand Down

0 comments on commit dfef227

Please sign in to comment.