Merge pull request #1791 from mightyiam/no-new-native-nonconstructor

feat!: no-new-native-nonconstructor

src/_intentionally-unused-rules.ts

@@ -17,6 +17,9 @@ export const intentionallyUnusedRules: string[] = [
   // Made redundant by 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-symbol',
+
   // 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.
   // >

src/rules.ts

@@ -323,7 +323,7 @@ export const eslintRules: Record<string, TSESLint.SharedConfig.RuleEntry> = {
   'no-multi-str': ['error'],
   'no-new': ['error'],
   'no-new-func': ['error'],
-  'no-new-symbol': ['error'],
+  'no-new-native-nonconstructor': ['error'],
   'no-new-wrappers': ['error'],
   'no-obj-calls': ['error'],
   'no-object-constructor': ['error'],

src/test/_expected-exported-value.ts

@@ -75,7 +75,7 @@ export const expectedEslintRules: Record<
   'no-multi-str': ['error'],
   'no-new': ['error'],
   'no-new-func': ['error'],
-  'no-new-symbol': ['error'],
+  'no-new-native-nonconstructor': ['error'],
   'no-new-wrappers': ['error'],
   'no-obj-calls': ['error'],
   'no-object-constructor': ['error'],

src/test/_rules_to_consider.ts

@@ -49,7 +49,6 @@ export const eslintRulesToConsider = [
   'no-multi-assign',
   'no-negated-condition',
   'no-nested-ternary',
-  'no-new-native-nonconstructor',
   'no-nonoctal-decimal-escape',
   'no-param-reassign',
   'no-plusplus',