Skip to content

Commit dffdf40

Browse files
authored
Prefer @typescript-eslint/no-unused-expression (#450)
- Update converter for no-unused-expression to use the @typescript-eslint variation
1 parent f81575a commit dffdf40

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

src/rules/converters/no-unused-expression.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ export const convertNoUnusedExpression: RuleConverter = (tslintRule) => {
44
return {
55
rules: [
66
{
7-
ruleName: "no-unused-expressions",
7+
ruleName: "@typescript-eslint/no-unused-expressions",
88
...collectNoticesAndArguments(tslintRule.ruleArguments),
99
},
1010
],

src/rules/converters/tests/no-unused-expression.test.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ describe(convertNoUnusedExpression, () => {
1212
notices: [
1313
`The TSLint optional config "allow-new" is the default ESLint behavior and will no longer be ignored.`,
1414
],
15-
ruleName: "no-unused-expressions",
15+
ruleName: "@typescript-eslint/no-unused-expressions",
1616
},
1717
],
1818
});
@@ -30,7 +30,7 @@ describe(convertNoUnusedExpression, () => {
3030
`The TSLint optional config "allow-new" is the default ESLint behavior and will no longer be ignored.`,
3131
],
3232
ruleArguments: [{ allowShortCircuit: true }],
33-
ruleName: "no-unused-expressions",
33+
ruleName: "@typescript-eslint/no-unused-expressions",
3434
},
3535
],
3636
});
@@ -45,7 +45,7 @@ describe(convertNoUnusedExpression, () => {
4545
rules: [
4646
{
4747
ruleArguments: [{ allowTaggedTemplates: true }],
48-
ruleName: "no-unused-expressions",
48+
ruleName: "@typescript-eslint/no-unused-expressions",
4949
},
5050
],
5151
});
@@ -60,7 +60,7 @@ describe(convertNoUnusedExpression, () => {
6060
rules: [
6161
{
6262
ruleArguments: [{ allowShortCircuit: true }],
63-
ruleName: "no-unused-expressions",
63+
ruleName: "@typescript-eslint/no-unused-expressions",
6464
},
6565
],
6666
});
@@ -75,7 +75,7 @@ describe(convertNoUnusedExpression, () => {
7575
rules: [
7676
{
7777
ruleArguments: [{ allowTaggedTemplates: true, allowShortCircuit: true }],
78-
ruleName: "no-unused-expressions",
78+
ruleName: "@typescript-eslint/no-unused-expressions",
7979
},
8080
],
8181
});

0 commit comments

Comments
 (0)