Skip to content

Commit 1aabfa2

Browse files
author
Josh Goldberg
authored
Corrected no-import-side-effect output rule (#271)
1 parent d81d262 commit 1aabfa2

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

src/rules/converters/no-import-side-effect.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,11 @@ export const convertNoImportSideEffect: RuleConverter = tsLintRule => {
44
return {
55
rules: [
66
{
7-
ruleName: "no-import-side-effect",
7+
plugins: ["eslint-plugin-import"],
8+
ruleName: "import/no-unassigned-import",
89
...(tsLintRule.ruleArguments.length !== 0 && {
910
notices: [
10-
"ESLint's no-import-side-effect now accepts a glob pattern for ignores; you'll need to manually convert your ignore-module settings.",
11+
"ESLint's import/no-unassigned-import now accepts a glob pattern for ignores; you'll need to manually convert your ignore-module settings.",
1112
],
1213
}),
1314
},

src/rules/converters/tests/no-import-side-effect.test.ts

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,8 @@ describe(convertNoImportSideEffect, () => {
99
expect(result).toEqual({
1010
rules: [
1111
{
12-
ruleName: "no-import-side-effect",
12+
plugins: ["eslint-plugin-import"],
13+
ruleName: "import/no-unassigned-import",
1314
},
1415
],
1516
});
@@ -23,10 +24,11 @@ describe(convertNoImportSideEffect, () => {
2324
expect(result).toEqual({
2425
rules: [
2526
{
26-
ruleName: "no-import-side-effect",
27+
plugins: ["eslint-plugin-import"],
2728
notices: [
28-
"ESLint's no-import-side-effect now accepts a glob pattern for ignores; you'll need to manually convert your ignore-module settings.",
29+
"ESLint's import/no-unassigned-import now accepts a glob pattern for ignores; you'll need to manually convert your ignore-module settings.",
2930
],
31+
ruleName: "import/no-unassigned-import",
3032
},
3133
],
3234
});

0 commit comments

Comments
 (0)