Skip to content

Commit 051c2c2

Browse files
committed
Disable a couple of annoying rules
1 parent 20b8ec1 commit 051c2c2

File tree

1 file changed

+10
-2
lines changed

1 file changed

+10
-2
lines changed

index.js

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -367,7 +367,11 @@ module.exports = {
367367
],
368368
'@typescript-eslint/no-namespace': 'error',
369369
'@typescript-eslint/no-non-null-asserted-optional-chain': 'error',
370-
'@typescript-eslint/no-non-null-assertion': 'error',
370+
371+
// Disabled for now. There are just too many places where you need to use it because of incorrect types, for example, the Node.js types.
372+
// TODO: Try to enable this again in 2023.
373+
// '@typescript-eslint/no-non-null-assertion': 'error',
374+
371375
'no-redeclare': 'off',
372376
'@typescript-eslint/no-redeclare': 'error',
373377
'@typescript-eslint/no-require-imports': 'error',
@@ -395,7 +399,11 @@ module.exports = {
395399
'@typescript-eslint/no-unnecessary-type-arguments': 'error',
396400
'@typescript-eslint/no-unnecessary-type-assertion': 'error',
397401
'@typescript-eslint/no-unnecessary-type-constraint': 'error',
398-
'@typescript-eslint/no-unsafe-argument': 'error',
402+
403+
// Disabled for now. When using try/catch, the error is by default `any` and if you pass the error anywhere, it will trigger this rule. This is a very common occurence.
404+
// TODO: Enable this rule when TypeScript has more strongly typed errors, probably 2023 at the earliest.
405+
// '@typescript-eslint/no-unsafe-argument': 'error',
406+
399407
'@typescript-eslint/no-unsafe-assignment': 'error',
400408
'@typescript-eslint/no-unsafe-call': 'error',
401409
'@typescript-eslint/no-unsafe-member-access': 'error',

0 commit comments

Comments
 (0)