Skip to content

Commit

Permalink
tools: auto fix custom eslint rule for lowercase-name-for-primitive.js
Browse files Browse the repository at this point in the history
1. Fixer for lowercase-name-for-primitive.js
2. Modified the node passed to fix it.

Refs : #16636
  • Loading branch information
shobhitchittora committed Dec 17, 2017
1 parent 1fa59b4 commit a389548
Showing 1 changed file with 14 additions and 4 deletions.
18 changes: 14 additions & 4 deletions tools/eslint-rules/lowercase-name-for-primitive.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,11 +22,11 @@ module.exports = function(context) {

switch (names.type) {
case 'Literal':
checkName(node, names.value);
checkName(names, names.value);
break;
case 'ArrayExpression':
names.elements.forEach((name) => {
checkName(node, name.value);
names.elements.forEach((name, index) => {
checkName(names.elements[index], name.value);
});
break;
}
Expand All @@ -36,8 +36,18 @@ module.exports = function(context) {
const lowercaseName = name.toLowerCase();
if (primitives.includes(lowercaseName) && !primitives.includes(name)) {
const msg = `primitive should use lowercase: ${name}`;
context.report(node, msg);
context.report({
node,
message: msg,
fix: (fixer) => {
return fixer.replaceText(
node,
`'${lowercaseName}'`
);
}
});
}

}

return {
Expand Down

0 comments on commit a389548

Please sign in to comment.