Skip to content

Commit 8dc8340

Browse files
skondrashovSasha Kondrashovkirkwaiblinger
authored
feat(eslint-plugin): add no-unnecessary-type-conversion to strict-type-checked ruleset (#11427)
add no-unnecessary-type-conversion to strict-type-checked ruleset Co-authored-by: Sasha Kondrashov <[email protected]> Co-authored-by: Kirk Waiblinger <[email protected]>
1 parent 956c1c7 commit 8dc8340

File tree

5 files changed

+5
-0
lines changed

5 files changed

+5
-0
lines changed

packages/eslint-plugin/src/configs/eslintrc/strict-type-checked-only.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ export = {
3030
'@typescript-eslint/no-unnecessary-template-expression': 'error',
3131
'@typescript-eslint/no-unnecessary-type-arguments': 'error',
3232
'@typescript-eslint/no-unnecessary-type-assertion': 'error',
33+
'@typescript-eslint/no-unnecessary-type-conversion': 'error',
3334
'@typescript-eslint/no-unnecessary-type-parameters': 'error',
3435
'@typescript-eslint/no-unsafe-argument': 'error',
3536
'@typescript-eslint/no-unsafe-assignment': 'error',

packages/eslint-plugin/src/configs/eslintrc/strict-type-checked.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ export = {
5151
'@typescript-eslint/no-unnecessary-type-arguments': 'error',
5252
'@typescript-eslint/no-unnecessary-type-assertion': 'error',
5353
'@typescript-eslint/no-unnecessary-type-constraint': 'error',
54+
'@typescript-eslint/no-unnecessary-type-conversion': 'error',
5455
'@typescript-eslint/no-unnecessary-type-parameters': 'error',
5556
'@typescript-eslint/no-unsafe-argument': 'error',
5657
'@typescript-eslint/no-unsafe-assignment': 'error',

packages/eslint-plugin/src/configs/flat/strict-type-checked-only.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@ export default (
4343
'@typescript-eslint/no-unnecessary-template-expression': 'error',
4444
'@typescript-eslint/no-unnecessary-type-arguments': 'error',
4545
'@typescript-eslint/no-unnecessary-type-assertion': 'error',
46+
'@typescript-eslint/no-unnecessary-type-conversion': 'error',
4647
'@typescript-eslint/no-unnecessary-type-parameters': 'error',
4748
'@typescript-eslint/no-unsafe-argument': 'error',
4849
'@typescript-eslint/no-unsafe-assignment': 'error',

packages/eslint-plugin/src/configs/flat/strict-type-checked.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,7 @@ export default (
6464
'@typescript-eslint/no-unnecessary-type-arguments': 'error',
6565
'@typescript-eslint/no-unnecessary-type-assertion': 'error',
6666
'@typescript-eslint/no-unnecessary-type-constraint': 'error',
67+
'@typescript-eslint/no-unnecessary-type-conversion': 'error',
6768
'@typescript-eslint/no-unnecessary-type-parameters': 'error',
6869
'@typescript-eslint/no-unsafe-argument': 'error',
6970
'@typescript-eslint/no-unsafe-assignment': 'error',

packages/eslint-plugin/src/rules/no-unnecessary-type-conversion.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ export default createRule<Options, MessageIds>({
2626
docs: {
2727
description:
2828
'Disallow conversion idioms when they do not change the type or value of the expression',
29+
recommended: 'strict',
2930
requiresTypeChecking: true,
3031
},
3132
hasSuggestions: true,

0 commit comments

Comments
 (0)