Skip to content

Commit fcee1c6

Browse files
buschtoenssindresorhus
authored andcommitted
Make the no-abusive-eslint-disable rule support shorthand syntax (#212)
1 parent 8306958 commit fcee1c6

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

rules/no-abusive-eslint-disable.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
'use strict';
22
const getDocsUrl = require('./utils/get-docs-url');
33

4-
const disableRegex = /^eslint-disable(-next-line|-line)?($|(\s+(@[\w-]+\/[\w-]+\/)?[\w-]+)?)/;
4+
const disableRegex = /^eslint-disable(-next-line|-line)?($|(\s+(@[\w-]+\/(?:[\w-]+\/)?)?[\w-]+)?)/;
55

66
const create = context => ({
77
Program: node => {

test/no-abusive-eslint-disable.js

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,8 @@ ruleTester.run('no-abusive-eslint-disable', rule, {
2525
'eval(); // eslint-disable-line plugin/rule',
2626
'eval(); // eslint-disable-line @scope/plugin/rule-name',
2727
'eval(); // eslint-disable-line no-eval, @scope/plugin/rule-name',
28+
'eval(); // eslint-disable-line @scope/rule-name',
29+
'eval(); // eslint-disable-line no-eval, @scope/rule-name',
2830
'eval(); // eslint-line-disable',
2931
'eval(); // some comment',
3032
`foo();
@@ -66,10 +68,6 @@ ruleTester.run('no-abusive-eslint-disable', rule, {
6668
{
6769
code: '// eslint-disable-next-line @scopewithoutplugin\neval();',
6870
errors: [error('Specify the rules you want to disable at line 1:0')]
69-
},
70-
{
71-
code: '// eslint-disable-next-line @scope/pluginwithoutrule\neval();',
72-
errors: [error('Specify the rules you want to disable at line 1:0')]
7371
}
7472
]
7573
});

0 commit comments

Comments
 (0)