Merge pull request #1646 from mightyiam/explicit-member-accessibility

chore: intentionally not use @typescript-eslint/explicit-member-accessibility

src/_intentionally-unused-rules.ts

@@ -3,4 +3,8 @@ export const intentionallyUnusedRules: string[] = [
   // > 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 trule 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',
 ]

src/test/_rules_to_consider.ts

@@ -1,6 +1,5 @@
 export const rulesToConsider = [
   '@typescript-eslint/default-param-last',
-  '@typescript-eslint/explicit-member-accessibility',
   '@typescript-eslint/explicit-module-boundary-types',
   '@typescript-eslint/init-declarations',
   '@typescript-eslint/lines-around-comment',