Skip to content

Commit b95ac67

Browse files
committed
⚒ disable ts/no-unnecessary-condition
This rule was problematic on my code in some situations. (Especially, for the variables that are initialized in beforeEach hook in test code.)
1 parent f460165 commit b95ac67

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

lib/configs/_override-ts.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,6 @@ module.exports = {
3939
"error",
4040
{ allowDestructuring: true },
4141
],
42-
"@mysticatea/ts/no-unnecessary-condition": "error",
4342
"@mysticatea/ts/no-unnecessary-qualifier": "error",
4443
"@mysticatea/ts/no-unnecessary-type-arguments": "error",
4544
"@mysticatea/ts/no-unnecessary-type-assertion": "error",
@@ -112,6 +111,7 @@ module.exports = {
112111
"@mysticatea/ts/no-namespace": "off", // I like the namespace for interfaces (type only things).
113112
"@mysticatea/ts/no-non-null-assertion": "off",
114113
"@mysticatea/ts/no-type-alias": "off",
114+
"@mysticatea/ts/no-unnecessary-condition": "off", // This was problematic for test code.
115115
"@mysticatea/ts/no-unused-vars": "off", // tsc verifies it.
116116
"@mysticatea/ts/no-use-before-define": "off", // tsc verifies it.
117117
"@mysticatea/ts/prefer-for-of": "off",

0 commit comments

Comments
 (0)