Skip to content

Commit 74e86c1

Browse files
committed
chore: fix ESLint errors
1 parent 34758df commit 74e86c1

17 files changed

+32
-27
lines changed

.eslintrc.yml

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,3 +2,18 @@ root: true
22
extends:
33
- plugin:@mysticatea/es2015
44
- plugin:@mysticatea/+eslint-plugin
5+
6+
overrides:
7+
- files: [ "lib/utils.js", "scripts/*.js" ]
8+
rules:
9+
"@mysticatea/node/no-sync": off
10+
11+
- files:
12+
[
13+
"lib/configs.js",
14+
"lib/foreign-rules/ts.js",
15+
"lib/processors.js",
16+
"lib/rules.js",
17+
]
18+
rules:
19+
"@mysticatea/node/global-require": off

tests/lib/configs/_rules.js

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ const {
99
ConfigArrayFactory,
1010
} = require("eslint/lib/cli-engine/config-array-factory")
1111
const Validator = require("eslint/lib/shared/config-validator")
12+
const { rules: removedRules } = require("eslint/conf/replacements.json")
1213
const { rules: PluginRulesIndex } = require("@mysticatea/eslint-plugin")
1314

1415
const coreRules = new Linter().getRules()
@@ -25,9 +26,7 @@ const deprecatedRuleNames = new Set(
2526
.filter(([, rule]) => rule && rule.meta && rule.meta.deprecated)
2627
.map(([ruleId]) => ruleId)
2728
)
28-
const removedRuleNames = new Set(
29-
Object.keys(require("eslint/conf/replacements.json").rules)
30-
)
29+
const removedRuleNames = new Set(Object.keys(removedRules))
3130

3231
const configFactory = new ConfigArrayFactory()
3332

tests/lib/configs/browser.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,10 @@
55
*/
66
"use strict"
77

8+
const config = require("../../../lib/configs/+browser")
89
const Rules = require("./_rules")
910

1011
describe("'+browser.js'", () => {
11-
const config = require("../../../lib/configs/+browser")
12-
1312
it("should be a valid config.", () => {
1413
Rules.validateConfig(config, "+browser.js")
1514
})

tests/lib/configs/es2015.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
"use strict"
77

88
const assert = require("assert")
9+
const config = require("../../../lib/configs/es2015")
910
const Rules = require("./_rules")
1011

1112
/**
@@ -22,7 +23,6 @@ function isES6Rule(ruleId) {
2223
}
2324

2425
describe("'es2015.js'", () => {
25-
const config = require("../../../lib/configs/es2015")
2626
const configuredRules = Rules.getRulesOfConfig(config, "es2015")
2727
const existingRules = Rules.getCoreRuleNames()
2828

tests/lib/configs/es2016.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,10 @@
55
*/
66
"use strict"
77

8+
const config = require("../../../lib/configs/es2016")
89
const Rules = require("./_rules")
910

1011
describe("'es2016.js'", () => {
11-
const config = require("../../../lib/configs/es2016")
12-
1312
it("should be a valid config.", () => {
1413
Rules.validateConfig(config, "es2016.js")
1514
})

tests/lib/configs/es2017.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,10 @@
55
*/
66
"use strict"
77

8+
const config = require("../../../lib/configs/es2017")
89
const Rules = require("./_rules")
910

1011
describe("'es2017.js'", () => {
11-
const config = require("../../../lib/configs/es2017")
12-
1312
it("should be a valid config.", () => {
1413
Rules.validateConfig(config, "es2017.js")
1514
})

tests/lib/configs/es2018.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,10 @@
55
*/
66
"use strict"
77

8+
const config = require("../../../lib/configs/es2018")
89
const Rules = require("./_rules")
910

1011
describe("'es2018.js'", () => {
11-
const config = require("../../../lib/configs/es2018")
12-
1312
it("should be a valid config.", () => {
1413
Rules.validateConfig(config, "es2018.js")
1514
})

tests/lib/configs/es2019.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,10 @@
44
*/
55
"use strict"
66

7+
const config = require("../../../lib/configs/es2019")
78
const Rules = require("./_rules")
89

910
describe("'es2019.js'", () => {
10-
const config = require("../../../lib/configs/es2019")
11-
1211
it("should be a valid config.", () => {
1312
Rules.validateConfig(config, "es2019.js")
1413
})

tests/lib/configs/es2020.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,10 @@
44
*/
55
"use strict"
66

7+
const config = require("../../../lib/configs/es2020")
78
const Rules = require("./_rules")
89

910
describe("'es2020.js'", () => {
10-
const config = require("../../../lib/configs/es2020")
11-
1211
it("should be a valid config.", () => {
1312
Rules.validateConfig(config, "es2020.js")
1413
})

tests/lib/configs/es5.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
"use strict"
77

88
const assert = require("assert")
9+
const config = require("../../../lib/configs/es5")
910
const Rules = require("./_rules")
1011

1112
/**
@@ -22,7 +23,6 @@ function isES6Rule(ruleId) {
2223
}
2324

2425
describe("'es5.js'", () => {
25-
const config = require("../../../lib/configs/es5")
2626
const configuredRules = Rules.getRulesOfConfig(config, "es5")
2727
const existingRules = [].concat(
2828
Rules.getCoreRuleNames(),

tests/lib/configs/eslint-plugin.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,10 @@
66
"use strict"
77

88
const assert = require("assert")
9+
const config = require("../../../lib/configs/+eslint-plugin")
910
const Rules = require("./_rules")
1011

1112
describe("'+eslint-plugin.js'", () => {
12-
const config = require("../../../lib/configs/+eslint-plugin")
1313
const configuredRules = Rules.getRulesOfConfig(config, "+eslint-plugin")
1414
const existingRules = Rules.getPluginRuleNames("eslint-plugin")
1515

tests/lib/configs/mocha.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,10 @@
55
*/
66
"use strict"
77

8+
const config = require("../../../lib/configs/_override-mocha")
89
const Rules = require("./_rules")
910

1011
describe("'mocha.js'", () => {
11-
const config = require("../../../lib/configs/_override-mocha")
12-
1312
it("should be a valid config.", () => {
1413
Rules.validateConfig(config, "mocha.js")
1514
})

tests/lib/configs/modules.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,10 @@
55
*/
66
"use strict"
77

8+
const config = require("../../../lib/configs/+modules")
89
const Rules = require("./_rules")
910

1011
describe("'+modules.js'", () => {
11-
const config = require("../../../lib/configs/+modules")
12-
1312
it("should be a valid config.", () => {
1413
Rules.validateConfig(config, "+modules.js")
1514
})

tests/lib/configs/node.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,10 @@
66
"use strict"
77

88
const assert = require("assert")
9+
const config = require("../../../lib/configs/+node")
910
const Rules = require("./_rules")
1011

1112
describe("'+node.js'", () => {
12-
const config = require("../../../lib/configs/+node")
1313
const configuredRules = Rules.getRulesOfConfig(config, "+node")
1414
const existingRules = Rules.getPluginRuleNames("node")
1515

tests/lib/configs/special.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,10 @@
55
*/
66
"use strict"
77

8+
const config = require("../../../lib/configs/_override-special")
89
const Rules = require("./_rules")
910

1011
describe("'special.js'", () => {
11-
const config = require("../../../lib/configs/_override-special")
12-
1312
it("should be a valid config.", () => {
1413
Rules.validateConfig(config, "special.js")
1514
})

tests/lib/configs/ts.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,10 @@
66
"use strict"
77

88
const assert = require("assert")
9+
const config = require("../../../lib/configs/_override-ts")
910
const Rules = require("./_rules")
1011

1112
describe("'ts.js'", () => {
12-
const config = require("../../../lib/configs/_override-ts")
1313
const configuredRules = Rules.getRulesOfConfig(config, "_override-ts")
1414
const existingRules = Rules.getPluginRuleNames("ts")
1515

tests/lib/configs/vue.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,10 @@
66
"use strict"
77

88
const assert = require("assert")
9+
const config = require("../../../lib/configs/_override-vue")
910
const Rules = require("./_rules")
1011

1112
describe("'vue.js'", () => {
12-
const config = require("../../../lib/configs/_override-vue")
1313
const configuredRules = Rules.getRulesOfConfig(config, "_override-ts")
1414
const existingRules = Rules.getPluginRuleNames("vue")
1515

0 commit comments

Comments
 (0)