diff --git a/src/_intentionally-unused-rules.ts b/src/_intentionally-unused-rules.ts index 5a20bfaa..89344c13 100644 --- a/src/_intentionally-unused-rules.ts +++ b/src/_intentionally-unused-rules.ts @@ -1,25 +1,33 @@ export const intentionallyUnusedRules: string[] = [ - // From rule documentation: - // > If possible, it is recommended to use tsconfig's noImplicitReturns option rather than this rule. noImplicitReturns is powered by TS's type information and control-flow analysis so it has better coverage than this rule. - '@typescript-eslint/consistent-return', - 'consistent-return', + // ## Incompatible with JS - // From trule documentation: + // From rule documentation: // > If you are working on a codebase within which you lint non-TypeScript code (i.e. `.js`/`.mjs`/`.cjs`/`.jsx`), you should ensure that you should use [ESLint `overrides`](https://eslint.org/docs/user-guide/configuring#disabling-rules-only-for-a-group-of-files) to only enable the rule on `.ts`/`.mts`/`.cts`/`.tsx` files. If you don't, then you will get unfixable lint errors reported within `.js`/`.mjs`/`.cjs`/`.jsx` files. '@typescript-eslint/explicit-member-accessibility', + // ## "too much" + '@typescript-eslint/default-param-last', 'default-param-last', - // style '@typescript-eslint/member-ordering', - // Made redundant by TS strict mode and `@typescript-eslint/explicit-function-return-type` + '@typescript-eslint/no-shadow', + 'no-shadow', + + // ## redundant + + // TS strict mode and `@typescript-eslint/explicit-function-return-type` '@typescript-eslint/explicit-module-boundary-types', - // Made redundant by no-new-native-nonconstructor + // no-new-native-nonconstructor 'no-new-symbol', + // From rule documentation: + // > If possible, it is recommended to use tsconfig's noImplicitReturns option rather than this rule. noImplicitReturns is powered by TS's type information and control-flow analysis so it has better coverage than this rule. + '@typescript-eslint/consistent-return', + 'consistent-return', + // From rule documentation: // > Requiring type annotations unnecessarily can be cumbersome to maintain and generally reduces code readability. TypeScript is often better at inferring types than easily written type annotations would allow. // > @@ -29,14 +37,15 @@ export const intentionallyUnusedRules: string[] = [ // Covered by strict TypeScript '@typescript-eslint/no-invalid-this', - // For project specific usage + // ## Project specific + '@typescript-eslint/no-restricted-imports', + '@typescript-eslint/no-restricted-types', + 'n/no-restricted-import', + 'n/no-restricted-require', - 'no-restricted-imports', - // Too strict - '@typescript-eslint/no-shadow', - 'no-shadow', + 'no-restricted-imports', ]