@modern-js-app/eslint-config 1.2.4 → 1.2.7

package/CHANGELOG.md

@@ -1,5 +1,33 @@
 # @modern-js-app/eslint-config
 
+## 1.2.7
+
+### Patch Changes
+
+- d6522e85: feat: remove max-params and max-statements rule
+- Updated dependencies [17d0cc46]
+  - @modern-js/babel-preset-app@1.2.6
+
+## 1.2.6
+
+### Patch Changes
+
+- 8e737893: feat: disable promise/prefer-await-to-then
+- 8e737893: feat: disable require-atomic-updates
+- a2761eb7: fix: exclude modern.config.ts from with type rules
+- 8e737893: feat: disable no-shadow
+- 8e737893: feat: disable @typescript-eslint/no-base-to-string
+- 8e737893: feat: disable filenames/match-exported
+- Updated dependencies [9d4a005b]
+  - @modern-js/babel-preset-app@1.2.5
+
+## 1.2.5
+
+### Patch Changes
+
+- 10f47286: fix: typescript eslint rule due to @typescript-eslint/eslint-plugin's breaking change
+- 0ef81796: feat: simplify react eslint rules
+
 ## 1.2.4
 
 ### Patch Changes

package/base.js

@@ -1,5 +1,4 @@
-/* eslint-disable max-lines, no-magic-numbers */
-
+/* eslint-disable max-lines */
 const { jsExtensions } = require('./utils');
 
 module.exports = {
@@ -97,7 +96,7 @@ module.exports = {
 
     /*
      * https://eslint.org/docs/rules/no-debugger
-     * @TIPS: for non-VSCode users, please use `debugger // eslint-disable-line`
+     * @TIPS: for non-VS Code users, please use `debugger // eslint-disable-line`
      *        instead of `debugger` to avoid it to be removed by IDE's autoFixOnSave feature
      */
     'no-debugger': 'error',
@@ -155,10 +154,6 @@ module.exports = {
     'no-unsafe-finally': 'error',
     // https://eslint.org/docs/rules/no-unsafe-negation
     'no-unsafe-negation': 'error',
-    // https://eslint.org/docs/rules/require-atomic-updates
-    // @bug https://github.com/eslint/eslint/issues/11899
-    'require-atomic-updates': 'warn',
-    // 'require-atomic-updates': 'error',
     // https://eslint.org/docs/rules/use-isnan
     'use-isnan': 'error',
     // https://eslint.org/docs/rules/valid-typeof
@@ -427,17 +422,6 @@ module.exports = {
       // @CUSTOM
     ],
 
-    /*
-     * https://eslint.org/docs/rules/no-shadow
-     * "no-shadow": "off",
-     */
-    'no-shadow': [
-      'error',
-      {
-        builtinGlobals: false,
-        allow: [],
-      },
-    ],
     // https://eslint.org/docs/rules/no-shadow-restricted-names
     'no-shadow-restricted-names': 'error',
     // https://eslint.org/docs/rules/no-undef
@@ -634,28 +618,11 @@ module.exports = {
         skipComments: true,
       },
     ],
-    // https://eslint.org/docs/rules/max-lines-per-function
-    'max-lines-per-function': 'off',
 
     /*
-     * 'max-lines-per-function': [
-     *   'error',
-     *   {
-     *     max: 50,
-     *     skipBlankLines: true,
-     *     skipComments: true,
-     *     IIFEs: true,
-     *   },
-     * ],
      * https://eslint.org/docs/rules/max-nested-callbacks
      */
     'max-nested-callbacks': ['warn', 4],
-    // https://eslint.org/docs/rules/max-params
-    'max-params': ['warn', 4],
-    // https://eslint.org/docs/rules/max-statements
-    'max-statements': ['warn', 20],
-    // https://eslint.org/docs/rules/max-statements-per-line
-    'max-statements-per-line': ['error', { max: 1 }],
     // https://eslint.org/docs/rules/multiline-comment-style
     // @TODO bug:
     // // class Foo {
@@ -1013,12 +980,6 @@ module.exports = {
     'react/button-has-type': 'error',
     // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/default-props-match-prop-types.md
     'react/default-props-match-prop-types': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/destructuring-assignment.md
-    'react/destructuring-assignment': [
-      'error',
-      'always',
-      { ignoreClassFields: true },
-    ],
 
     /*
      * https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/display-name.md
@@ -1042,59 +1003,6 @@ module.exports = {
      * - 嗯ok，这个规则现在的性价比是不太高了…
      */
     'react/display-name': 'off',
-    // 'react/display-name': [
-    //   'error',
-    //   {
-    //     ignoreTranspilerName: false,
-    //   },
-    // ],
-    // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/forbid-component-props.md
-    'react/forbid-component-props': 'off',
-
-    /*
-     * @CUSTOM
-     * "react/forbid-component-props": [
-     *   "error",
-     *   {
-     *     "forbid": [
-     *       "className",
-     *       "style"
-     *     ]
-     *   }
-     * ],
-     * https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/forbid-dom-props.md
-     */
-    'react/forbid-dom-props': 'off',
-
-    /*
-     * @CUSTOM
-     * "react/forbid-dom-props": [
-     *   "error",
-     *   {
-     *     "forbid": []
-     *   }
-     * ],
-     * https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/forbid-elements.md
-     */
-    'react/forbid-elements': 'off',
-
-    /*
-     * @CUSTOM
-     * "react/forbid-elements": [
-     *   "error",
-     *   {
-     *     "forbid": [
-     *       {
-     *         "element": "button",
-     *         "message": "use <Button> instead"
-     *       }
-     *     ]
-     *   }
-     * ],
-     * https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/forbid-prop-types.md
-     */
-    'react/forbid-prop-types': 'off',
-
     /*
      * @CUSTOM
      * "react/forbid-prop-types": [
@@ -1118,52 +1026,22 @@ module.exports = {
     'react/no-access-state-in-setstate': 'error',
     // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-array-index-key.md
     'react/no-array-index-key': 'warn',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-children-prop.md
-    'react/no-children-prop': 'error',
-
     /*
      * https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-danger.md
      * @TIPS
      * explicitly declare `eslint-disable` when truly necessary
      */
     'react/no-danger': 'error',
-
-    /*
-     * https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-danger-with-children.md
-     * @TIPS
-     * explicitly declare `eslint-disable` when truly necessary
-     */
-    'react/no-danger-with-children': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-deprecated.md
-    'react/no-deprecated': 'error',
     // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-did-mount-set-state.md
     'react/no-did-mount-set-state': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-did-update-set-state.md
-    'react/no-did-update-set-state': 'off',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-direct-mutation-state.md
-    'react/no-direct-mutation-state': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-find-dom-node.md
-    'react/no-find-dom-node': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-is-mounted.md
-    'react/no-is-mounted': 'error',
     // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-multi-comp.md
     'react/no-multi-comp': ['error', { ignoreStateless: true }],
     // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-redundant-should-component-update.md
     'react/no-redundant-should-component-update': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-render-return-value.md
-    'react/no-render-return-value': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-set-state.md
-    'react/no-set-state': 'off',
     // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-typos.md
     'react/no-typos': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-string-refs.md
-    'react/no-string-refs': 'error',
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/no-this-in-sfc.md
     'react/no-this-in-sfc': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-unescaped-entities.md
-    'react/no-unescaped-entities': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-unknown-property.md
-    'react/no-unknown-property': 'error',
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/no-unsafe.md
     'react/no-unsafe': ['error', { checkAliases: true }],
     // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/no-unused-prop-types.md
@@ -1194,14 +1072,6 @@ module.exports = {
       { forbidDefaultForRequired: true },
     ],
 
-    /*
-     * https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/require-optimization.md
-     * https://github.com/facebook/react/pull/7195#issuecomment-236361372
-     * https://hackernoon.com/react-purecomponent-considered-harmful-8155b5c1d4bc
-     */
-    'react/require-optimization': 'off',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/require-render-return.md
-    'react/require-render-return': 'error',
     // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/self-closing-comp.md
     'react/self-closing-comp': [
       'error',
@@ -1210,102 +1080,15 @@ module.exports = {
         html: false,
       },
     ],
-    // https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/sort-comp.md
-    'react/sort-comp': [
-      'error',
-      {
-        order: [
-          'type-annotations',
-          'class-properties-and-methods',
-          'instance-variables',
-          'state',
-          'getInitialState',
-          'getChildContext',
-          'getDefaultProps',
-          'constructor',
-          'lifecycle',
-          '/^is.+$/',
-          '/^_is.+$/',
-          '/^has.+$/',
-          '/^_has.+$/',
-          '/^should.+$/',
-          '/^_should.+$/',
-          '/^get.+$/',
-          '/^_get.+$/',
-          'getters',
-          '/^set.+$/',
-          '/^_set.+$/',
-          'setters',
-          '/^on.+$/',
-          '/^_on.+$/',
-          '/^handle.+$/',
-          '/^_handle.+$/',
-          'instance-methods',
-          'everything-else',
-          'rendering',
-        ],
-        groups: {
-          'class-properties-and-methods': [
-            'displayName',
-            'propTypes',
-            'contextTypes',
-            'childContextTypes',
-            'mixins',
-            'statics',
-            'defaultProps',
-            'getDerivedStateFromProps',
-            'static-methods',
-          ],
-          lifecycle: [
-            'UNSAFE_componentWillMount',
-            'componentWillMount',
-            'componentDidMount',
-            'UNSAFE_componentWillReceiveProps',
-            'componentWillReceiveProps',
-            'shouldComponentUpdate',
-            'getSnapshotBeforeUpdate',
-            'UNSAFE_componentWillUpdate',
-            'componentWillUpdate',
-            'componentDidUpdate',
-            'componentWillUnmount',
-          ],
-          rendering: ['/^render.+$/', 'render'],
-        },
-      },
-    ],
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/sort-prop-types.md
-    'react/sort-prop-types': 'off',
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/style-prop-object.md
     'react/style-prop-object': 'error',
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/void-dom-elements-no-children.md
     'react/void-dom-elements-no-children': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-boolean-value.md
-    'react/jsx-boolean-value': ['error', 'always'],
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-child-element-spacing.md
-    'react/jsx-child-element-spacing': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-closing-bracket-location.md
-    'react/jsx-closing-bracket-location': [
-      'error',
-      {
-        // "selfClosing": "props-aligned",
-        selfClosing: 'after-props',
-        // "nonEmpty": "props-aligned"
-        nonEmpty: 'after-props',
-      },
-    ],
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-closing-tag-location.md
-    'react/jsx-closing-tag-location': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-curly-spacing.md
-    'react/jsx-curly-spacing': ['error', { when: 'never' }],
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-equals-spacing.md
-    'react/jsx-equals-spacing': ['error', 'never'],
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-filename-extension.md
     'react/jsx-filename-extension': [
       'error',
       { extensions: ['.jsx', '.tsx', '.mjsx', '.cjsx'] },
     ],
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-first-prop-new-line.md
-    'react/jsx-first-prop-new-line': ['error', 'multiline'],
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-handler-names.md
     'react/jsx-handler-names': [
       'error',
@@ -1315,22 +1098,6 @@ module.exports = {
         eventHandlerPropPrefix: 'on',
       },
     ],
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-indent.md
-    'react/jsx-indent': ['error', 2],
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-indent-props.md
-    'react/jsx-indent-props': ['error', 2],
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-key.md
-    'react/jsx-key': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-max-depth.md
-    'react/jsx-max-depth': 'off',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-max-props-per-line.md
-    'react/jsx-max-props-per-line': [
-      'error',
-      {
-        maximum: 1,
-        when: 'always',
-      },
-    ],
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-no-bind.md
     'react/jsx-no-bind': [
       'error',
@@ -1339,85 +1106,20 @@ module.exports = {
         allowArrowFunctions: true,
       },
     ],
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-no-comment-textnodes.md
-    'react/jsx-no-comment-textnodes': 'error',
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-no-duplicate-props.md
     'react/jsx-no-duplicate-props': ['error', { ignoreCase: true }],
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-no-literals.md
-    'react/jsx-no-literals': 'off',
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-no-target-blank.md
     'react/jsx-no-target-blank': 'off',
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-no-undef.md
     'react/jsx-no-undef': ['error', { allowGlobals: true }],
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-one-expression-per-line.md
-    'react/jsx-one-expression-per-line': ['error', { allow: 'single-child' }],
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-curly-brace-presence.md
-    'react/jsx-curly-brace-presence': 'off',
-
-    /*
-     * "react/jsx-curly-brace-presence": [
-     *   "error",
-     *   "never"
-     * ],
-     * https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-fragments.md
-     * @TODO bug
-     */
-    'react/jsx-fragments': 'off', // ['error', 'syntax'],
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-pascal-case.md
     'react/jsx-pascal-case': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-props-no-multi-spaces.md
-    'react/jsx-props-no-multi-spaces': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-sort-default-props.md
-    'react/jsx-sort-default-props': 'off',
-
-    /*
-     * https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-sort-props.md
-     * @TODO bug: conflict with prettier
-     */
-    'react/jsx-sort-props': 'off',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-tag-spacing.md
-    'react/jsx-tag-spacing': [
-      'error',
-      {
-        closingSlash: 'never',
-        beforeSelfClosing: 'always',
-        afterOpening: 'never',
-        beforeClosing: 'never',
-      },
-    ],
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-uses-react.md
-    // react 17
-    'react/jsx-uses-react': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-uses-vars.md
-    'react/jsx-uses-vars': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-wrap-multilines.md
-    'react/jsx-wrap-multilines': [
-      'error',
-      {
-        declaration: true,
-        assignment: true,
-        return: true,
-        arrow: true,
-        condition: 'parens-new-line',
-        logical: 'parens-new-line',
-        prop: 'parens-new-line',
-      },
-    ],
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/no-unused-state.md
     'react/no-unused-state': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-props-no-spreading.md
-    'react/jsx-props-no-spreading': 'off',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/prefer-read-only-props.md
-    'react/prefer-read-only-props': 'off',
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/state-in-constructor.md
     'react/state-in-constructor': ['error', 'never'],
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/static-property-placement.md
     'react/static-property-placement': 'error',
-    // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-curly-newline.md
-    'react/jsx-curly-newline': [
-      'error',
-      { multiline: 'require', singleline: 'forbid' },
-    ],
     // https://reactjs.org/docs/hooks-rules.html
     'react-hooks/rules-of-hooks': 'error',
     // https://github.com/facebook/react/issues/16006
@@ -1628,8 +1330,6 @@ module.exports = {
      * use Camel Case for others
      */
     'filenames/match-regex': ['error', '^[\\[\\]_a-zA-Z0-9.-]+$'],
-    // https://www.npmjs.com/package/eslint-plugin-filenames#matching-exported-values-match-exported
-    'filenames/match-exported': ['error', ['kebab', 'camel', 'pascal']],
     // https://www.npmjs.com/package/eslint-plugin-filenames#dont-allow-indexjs-files-no-index
     'filenames/no-index': 'off',
 
@@ -1659,12 +1359,6 @@ module.exports = {
     // https://github.com/xjamundx/eslint-plugin-promise/blob/master/docs/rules/valid-params.md
     'promise/valid-params': 'error',
 
-    /*
-     * https://github.com/xjamundx/eslint-plugin-promise/blob/master/docs/rules/prefer-await-to-then.md
-     * @CUSTOM
-     */
-    'promise/prefer-await-to-then': 'error',
-
     /*
      * https://github.com/xjamundx/eslint-plugin-promise/blob/master/docs/rules/prefer-await-to-callbacks.md
      * @CUSTOM
@@ -1829,7 +1523,6 @@ module.exports = {
       files: ['**/pages/**/_*', '**/pages/**/index.*', '**/pages/**/\\[**'],
       rules: {
         'filenames/match-regex': 'off',
-        'filenames/match-exported': 'off',
       },
     },
     {
@@ -1849,4 +1542,4 @@ module.exports = {
     },
   ],
 };
-/* eslint-enable max-lines, no-magic-numbers */
+/* eslint-enable max-lines */

package/index.js

@@ -1,4 +1,4 @@
 module.exports = {
   // https://eslint.org/docs/user-guide/configuring#extending-configuration-files
-  extends: ['./base.js', './ts.withType.js', './prettier'],
+  extends: ['./base.js', './ts', './ts.withType.js', './prettier'],
 };

package/jest.config.js

@@ -2,7 +2,6 @@ const sharedConfig = require('@scripts/jest-config');
 
 /** @type {import('@jest/types').Config.InitialOptions} */
 module.exports = {
-  // eslint-disable-next-line node/no-unsupported-features/es-syntax
   ...sharedConfig,
   rootDir: __dirname,
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@modern-js-app/eslint-config",
-  "version": "1.2.4",
+  "version": "1.2.7",
   "description": "The meta-framework suite designed from scratch for frontend-focused modern web development.",
   "homepage": "https://modernjs.dev",
   "bugs": "https://github.com/modern-js-dev/modern.js/issues",
@@ -15,8 +15,8 @@
   "peerDependencies": {
     "@babel/eslint-parser": "^7.15.0",
     "@babel/eslint-plugin": "^7.13.10",
-    "@typescript-eslint/eslint-plugin": "^5.12.1",
-    "@typescript-eslint/parser": "^5.12.1",
+    "@typescript-eslint/eslint-plugin": "^5.17.0",
+    "@typescript-eslint/parser": "^5.17.0",
     "eslint": "^7.32.0",
     "eslint-config-prettier": "^8.3.0",
     "eslint-import-resolver-webpack": "^0.13.1",
@@ -32,7 +32,7 @@
     "prettier": "^2.5.1"
   },
   "dependencies": {
-    "@modern-js/babel-preset-app": "^1.2.0"
+    "@modern-js/babel-preset-app": "^1.2.6"
   },
   "devDependencies": {
     "eslint": "^7.32.0",

package/ts.js

@@ -165,7 +165,21 @@ module.exports = {
             leadingUnderscore: 'allowSingleOrDouble',
             trailingUnderscore: 'allowSingleOrDouble',
           },
-
+          // https://typescript-eslint.io/rules/naming-convention/#ignore-properties-that-require-quotes
+          {
+            selector: [
+              'classProperty',
+              'objectLiteralProperty',
+              'typeProperty',
+              'classMethod',
+              'objectLiteralMethod',
+              'typeMethod',
+              'accessor',
+              'enumMember',
+            ],
+            format: null,
+            modifiers: ['requiresQuotes'],
+          },
           {
             selector: 'variable',
             format: ['camelCase', 'PascalCase', 'UPPER_CASE', 'snake_case'],
@@ -376,15 +390,6 @@ module.exports = {
         // https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-redeclare.md
         'no-redeclare': 'off',
         '@typescript-eslint/no-redeclare': ['error', { builtinGlobals: true }],
-        // https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-shadow.md
-        'no-shadow': 'off',
-        '@typescript-eslint/no-shadow': [
-          'error',
-          {
-            builtinGlobals: false,
-            allow: [],
-          },
-        ],
 
         // https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/space-before-function-paren.md
         'space-before-function-paren': 'off',

package/ts.withType.js

@@ -2,7 +2,8 @@ module.exports = {
   overrides: [
     {
       files: ['*.ts', '*.d.ts', '*.tsx'],
-      extends: ['./ts'],
+      // modern.config.ts is usually not included in tsconfig.json
+      excludedFiles: ['modern.config.ts'],
       parserOptions: {
         ecmaFeatures: { jsx: true },
         // don't set tsconfigRootDir, using the path relative to the cwd
@@ -23,11 +24,6 @@ module.exports = {
         ],
         // https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-unnecessary-type-assertion.md
         '@typescript-eslint/no-unnecessary-type-assertion': 'error',
-        // https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/restrict-template-expressions.md
-        '@typescript-eslint/restrict-template-expressions': [
-          'error',
-          { allowNumber: true, allowAny: true },
-        ],
         // https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-base-to-string.md
         '@typescript-eslint/no-base-to-string': 'error',
         // https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-unnecessary-boolean-literal-compare.md