feat(eslint-plugin): final final config changes for v6 (typescript-eslint#7157)

JoshuaKGoldberg · web-flow · commit e35c5c1c39f3 · 2023-07-04T09:52:38.000-07:00
* feat(eslint-plugin): final final config changes for v6

* yarn generate:configs

* Fix unused lint suppression complaint and test snapshot

* unknown, not any, in recommended-does-not-require-program

* Updated endColumn
diff --git a/packages/eslint-plugin/src/configs/recommended-type-checked.ts b/packages/eslint-plugin/src/configs/recommended-type-checked.ts
@@ -16,6 +16,7 @@ export = {
     '@typescript-eslint/no-base-to-string': 'error',
     '@typescript-eslint/no-duplicate-enum-values': 'error',
     '@typescript-eslint/no-duplicate-type-constituents': 'error',
+    '@typescript-eslint/no-explicit-any': 'error',
     '@typescript-eslint/no-extra-non-null-assertion': 'error',
     '@typescript-eslint/no-floating-promises': 'error',
     '@typescript-eslint/no-for-in-array': 'error',
diff --git a/packages/eslint-plugin/src/configs/recommended.ts b/packages/eslint-plugin/src/configs/recommended.ts
@@ -13,6 +13,7 @@ export = {
     'no-array-constructor': 'off',
     '@typescript-eslint/no-array-constructor': 'error',
     '@typescript-eslint/no-duplicate-enum-values': 'error',
+    '@typescript-eslint/no-explicit-any': 'error',
     '@typescript-eslint/no-extra-non-null-assertion': 'error',
     'no-loss-of-precision': 'off',
     '@typescript-eslint/no-loss-of-precision': 'error',
diff --git a/packages/eslint-plugin/src/configs/strict-type-checked.ts b/packages/eslint-plugin/src/configs/strict-type-checked.ts
@@ -14,6 +14,7 @@ export = {
     'no-array-constructor': 'off',
     '@typescript-eslint/no-array-constructor': 'error',
     '@typescript-eslint/no-base-to-string': 'error',
+    '@typescript-eslint/no-confusing-void-expression': 'error',
     '@typescript-eslint/no-duplicate-enum-values': 'error',
     '@typescript-eslint/no-duplicate-type-constituents': 'error',
     '@typescript-eslint/no-dynamic-delete': 'error',
diff --git a/packages/eslint-plugin/src/configs/stylistic-type-checked.ts b/packages/eslint-plugin/src/configs/stylistic-type-checked.ts
@@ -19,7 +19,6 @@ export = {
     'dot-notation': 'off',
     '@typescript-eslint/dot-notation': 'error',
     '@typescript-eslint/no-confusing-non-null-assertion': 'error',
-    '@typescript-eslint/no-confusing-void-expression': 'error',
     'no-empty-function': 'off',
     '@typescript-eslint/no-empty-function': 'error',
     '@typescript-eslint/no-empty-interface': 'error',
@@ -31,6 +30,5 @@ export = {
     '@typescript-eslint/prefer-nullish-coalescing': 'error',
     '@typescript-eslint/prefer-optional-chain': 'error',
     '@typescript-eslint/prefer-string-starts-ends-with': 'error',
-    '@typescript-eslint/sort-type-constituents': 'error',
   },
 };
diff --git a/packages/eslint-plugin/src/configs/stylistic.ts b/packages/eslint-plugin/src/configs/stylistic.ts
@@ -25,6 +25,5 @@ export = {
     '@typescript-eslint/prefer-function-type': 'error',
     '@typescript-eslint/prefer-namespace-keyword': 'error',
     '@typescript-eslint/prefer-optional-chain': 'error',
-    '@typescript-eslint/sort-type-constituents': 'error',
   },
 };
diff --git a/packages/eslint-plugin/src/rules/no-confusing-void-expression.ts b/packages/eslint-plugin/src/rules/no-confusing-void-expression.ts
@@ -28,7 +28,7 @@ export default util.createRule<Options, MessageId>({
     docs: {
       description:
         'Require expressions of type void to appear in statement position',
-      recommended: 'stylistic',
+      recommended: 'strict',
       requiresTypeChecking: true,
     },
     messages: {
diff --git a/packages/eslint-plugin/src/rules/no-explicit-any.ts b/packages/eslint-plugin/src/rules/no-explicit-any.ts
@@ -17,7 +17,7 @@ export default util.createRule<Options, MessageIds>({
     type: 'suggestion',
     docs: {
       description: 'Disallow the `any` type',
-      recommended: 'strict',
+      recommended: 'recommended',
     },
     fixable: 'code',
     hasSuggestions: true,
diff --git a/packages/eslint-plugin/src/rules/sort-type-constituents.ts b/packages/eslint-plugin/src/rules/sort-type-constituents.ts
@@ -112,7 +112,6 @@ export default util.createRule<Options, MessageIds>({
     docs: {
       description:
         'Enforce constituents of a type union/intersection to be sorted alphabetically',
-      recommended: 'stylistic',
     },
     fixable: 'code',
     hasSuggestions: true,
diff --git a/packages/eslint-plugin/src/rules/strict-boolean-expressions.ts b/packages/eslint-plugin/src/rules/strict-boolean-expressions.ts
@@ -145,7 +145,7 @@ export default util.createRule<Options, MessageId>({
       allowNullableBoolean: false,
       allowNullableString: false,
       allowNullableNumber: false,
-      allowNullableEnum: true,
+      allowNullableEnum: false,
       allowAny: false,
       allowRuleToRunWithoutStrictNullChecksIKnowWhatIAmDoing: false,
     },
diff --git a/packages/integration-tests/fixtures/recommended-does-not-require-program/index.ts b/packages/integration-tests/fixtures/recommended-does-not-require-program/index.ts
@@ -1 +1 @@
-var foo: any = true;
+var foo: unknown = true;
diff --git a/packages/integration-tests/tests/__snapshots__/recommended-does-not-require-program.test.ts.snap b/packages/integration-tests/tests/__snapshots__/recommended-does-not-require-program.test.ts.snap
@@ -11,7 +11,7 @@ exports[`recommended-does-not-require-program should lint successfully 1`] = `
     "messages": [
       {
         "column": 7,
-        "endColumn": 15,
+        "endColumn": 19,
         "endLine": 1,
         "line": 1,
         "message": "'foo' is assigned a value but never used.",
@@ -21,7 +21,7 @@ exports[`recommended-does-not-require-program should lint successfully 1`] = `
         "severity": 2,
       },
     ],
-    "output": "const foo: any = true;
+    "output": "const foo: unknown = true;
 ",
     "suppressedMessages": [],
     "usedDeprecatedRules": [],
diff --git a/packages/typescript-estree/src/ts-estree/ts-nodes.ts b/packages/typescript-estree/src/ts-estree/ts-nodes.ts
@@ -21,7 +21,6 @@ declare module 'typescript' {
 
 export type TSToken = ts.Token<ts.SyntaxKind>;
 
-/* eslint-disable @typescript-eslint/sort-type-constituents */
 export type TSNode =
   | ts.AssertClause
   | ts.AssertEntry
@@ -216,4 +215,3 @@ export type TSNode =
   | ts.JSDocOptionalType
   | ts.JSDocVariadicType
   | ts.JSDocAuthorTag;
-/* eslint-enable @typescript-eslint/sort-type-constituents */

Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-var foo: any = true;`
	`1`	`+var foo: unknown = true;`