Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit ccfebdf

Browse files
koddssonkoddsson
committed
Fix violations
1 parent 5fd7271 commit ccfebdf

12 files changed

+31
-28
lines changed

‎lib/rules/array-foreach.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ module.exports = {
1212
return {
1313
CallExpression(node) {
1414
if (node.callee.property && node.callee.property.name === 'forEach') {
15-
context.report(node, 'Prefer for...of instead of Array.forEach')
15+
context.report({node, message: 'Prefer for...of instead of Array.forEach'})
1616
}
1717
}
1818
}

‎lib/rules/async-currenttarget.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ module.exports = {
1919
if (node.property && node.property.name === 'currentTarget') {
2020
const scope = context.getScope()
2121
if (scope.block.async && scopeDidWait.has(scope)) {
22-
context.report(node, 'event.currentTarget inside an async function is error prone')
22+
context.report({node, message: 'event.currentTarget inside an async function is error prone'})
2323
}
2424
}
2525
}

‎lib/rules/async-preventdefault.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ module.exports = {
1919
if (node.callee.property && node.callee.property.name === 'preventDefault') {
2020
const scope = context.getScope()
2121
if (scope.block.async && scopeDidWait.has(scope)) {
22-
context.report(node, 'event.preventDefault() inside an async function is error prone')
22+
context.report({node, message: 'event.preventDefault() inside an async function is error prone'})
2323
}
2424
}
2525
}

‎lib/rules/authenticity-token.js

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,11 @@ module.exports = {
1111
create(context) {
1212
function checkAuthenticityTokenUsage(node, str) {
1313
if (str.includes('authenticity_token')) {
14-
context.report(
14+
context.report({
1515
node,
16-
'Form CSRF tokens (authenticity tokens) should not be created in JavaScript and their values should not be used directly for XHR requests.'
17-
)
16+
message:
17+
'Form CSRF tokens (authenticity tokens) should not be created in JavaScript and their values should not be used directly for XHR requests.'
18+
})
1819
}
1920
}
2021

‎lib/rules/js-class-name.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,14 +17,14 @@ module.exports = {
1717
const matches = str.match(allJsClassNameRegexp) || []
1818
for (const match of matches) {
1919
if (!match.match(validJsClassNameRegexp)) {
20-
context.report(node, 'js- class names should be lowercase and only contain dashes.')
20+
context.report({node, message: 'js- class names should be lowercase and only contain dashes.'})
2121
}
2222
}
2323
}
2424

2525
function checkStringEndsWithJSClassName(node, str) {
2626
if (str.match(endWithJsClassNameRegexp)) {
27-
context.report(node, 'js- class names should be statically defined.')
27+
context.report({node, message: 'js- class names should be statically defined.'})
2828
}
2929
}
3030

‎lib/rules/no-blur.js

Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,18 @@
1-
module.exports = function (context) {
2-
return {
3-
meta: {
4-
type: 'problem',
5-
docs: {
6-
description: 'disallow usage of `Element.prototype.blur()`',
7-
url: require('../url')(module)
8-
},
9-
schema: []
1+
module.exports = {
2+
meta: {
3+
type: 'problem',
4+
docs: {
5+
description: 'disallow usage of `Element.prototype.blur()`',
6+
url: require('../url')(module)
107
},
11-
CallExpression(node) {
12-
if (node.callee.property && node.callee.property.name === 'blur') {
13-
context.report(node, 'Do not use element.blur(), instead restore the focus of a previous element.')
8+
schema: []
9+
},
10+
create(context) {
11+
return {
12+
CallExpression(node) {
13+
if (node.callee.property && node.callee.property.name === 'blur') {
14+
context.report({node, message: 'Do not use element.blur(), instead restore the focus of a previous element.'})
15+
}
1416
}
1517
}
1618
}

‎lib/rules/no-dataset.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ module.exports = {
1212
return {
1313
MemberExpression(node) {
1414
if (node.property && node.property.name === 'dataset') {
15-
context.report(node, "Use getAttribute('data-your-attribute') instead of dataset.")
15+
context.report({node, message: "Use getAttribute('data-your-attribute') instead of dataset."})
1616
}
1717
}
1818
}

‎lib/rules/no-implicit-buggy-globals.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ module.exports = {
2525
(def.type === 'Variable' && def.parent.kind === 'const') ||
2626
(def.type === 'Variable' && def.parent.kind === 'let')
2727
) {
28-
context.report(def.node, 'Implicit global variable, assign as global property instead.')
28+
context.report({node: def.node, message: 'Implicit global variable, assign as global property instead.'})
2929
}
3030
}
3131
}

‎lib/rules/no-then.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,9 +12,9 @@ module.exports = {
1212
return {
1313
MemberExpression(node) {
1414
if (node.property && node.property.name === 'then') {
15-
context.report(node.property, 'Prefer async/await to Promise.then()')
15+
context.report({node: node.property, message: 'Prefer async/await to Promise.then()'})
1616
} else if (node.property && node.property.name === 'catch') {
17-
context.report(node.property, 'Prefer async/await to Promise.catch()')
17+
context.report({node: node.property, message: 'Prefer async/await to Promise.catch()'})
1818
}
1919
}
2020
}

‎lib/rules/no-useless-passive.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ module.exports = {
1515

1616
create(context) {
1717
return {
18-
['CallExpression[callee.property.name="addEventListener"]']: function (node) {
18+
['CallExpression[callee.property.name="addEventListener"]']: function(node) {
1919
const [name, listener, options] = node.arguments
2020
if (name.type !== 'Literal') return
2121
if (passiveEventListenerNames.has(name.value)) return

‎lib/rules/prefer-observers.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ module.exports = {
1414

1515
create(context) {
1616
return {
17-
['CallExpression[callee.property.name="addEventListener"]']: function (node) {
17+
['CallExpression[callee.property.name="addEventListener"]']: function(node) {
1818
const [name] = node.arguments
1919
if (name.type !== 'Literal') return
2020
if (!(name.value in observerMap)) return

‎lib/rules/require-passive-events.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,13 +14,13 @@ module.exports = {
1414

1515
create(context) {
1616
return {
17-
['CallExpression[callee.property.name="addEventListener"]']: function (node) {
17+
['CallExpression[callee.property.name="addEventListener"]']: function(node) {
1818
const [name, listener, options] = node.arguments
1919
if (!listener) return
2020
if (name.type !== 'Literal') return
2121
if (!passiveEventListenerNames.has(name.value)) return
2222
if (options && options.type === 'ObjectExpression' && options.properties.some(propIsPassiveTrue)) return
23-
context.report(node, `High Frequency Events like "${name.value}" should be \`passive: true\``)
23+
context.report({node, message: `High Frequency Events like "${name.value}" should be \`passive: true\``})
2424
}
2525
}
2626
}

0 commit comments

Comments
 (0)