feat: support ESLint 8.x

.eslintrc.js

@@ -1,5 +1,7 @@
 'use strict';
 
+const { version } = require('eslint/package.json');
+const semver = require('semver');
 const globals = require('./src/globals.json');
 
 module.exports = {
@@ -28,12 +30,18 @@ module.exports = {
     es6: true,
   },
   rules: {
+    // eslint-plugin-prettier removed ESLint v6 support
+    ...(semver.gte(version, '7.28.0') ? {} : { 'prettier/prettier': 'off' }),
     '@typescript-eslint/array-type': ['error', { default: 'array-simple' }],
     '@typescript-eslint/no-require-imports': 'error',
     '@typescript-eslint/ban-ts-comment': 'warn',
     '@typescript-eslint/ban-types': 'error',
     '@typescript-eslint/no-unused-vars': 'error',
     'eslint-comments/no-unused-disable': 'error',
+    'eslint-plugin/require-meta-fixable': [
+      'error',
+      { catchNoFixerButFixableProperty: true },
+    ],
     'eslint-plugin/test-case-property-ordering': 'error',
     'no-else-return': 'error',
     'no-negated-condition': 'error',

.github/workflows/nodejs.yml

@@ -20,7 +20,7 @@ jobs:
 
       - uses: actions/setup-node@v2
         with:
-          node-version: 14.x
+          node-version: 16
           cache: yarn
 
       - name: Validate cache
@@ -38,7 +38,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v2
         with:
-          node-version: 14.x
+          node-version: 16
           cache: yarn
       - name: install
         run: yarn
@@ -52,7 +52,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v2
         with:
-          node-version: 14.x
+          node-version: 16
           cache: yarn
       - name: install
         run: yarn
@@ -67,9 +67,12 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        node-version: [12.x, 14.x, 16.x]
-        eslint-version: [6, 7]
+        node-version: [12, 14, 16]
+        eslint-version: [6, 7, 8]
         ts-eslint-plugin-version: [4, 5]
+        exclude:
+          - eslint-version: 8
+            ts-eslint-plugin-version: 4
     runs-on: ubuntu-latest
 
     steps:
@@ -104,7 +107,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v2
         with:
-          node-version: 14.x
+          node-version: 16
           cache: yarn
       - name: install
         run: yarn
@@ -123,7 +126,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v2
         with:
-          node-version: 14.x
+          node-version: 16
           cache: yarn
       - name: install
         run: yarn
@@ -146,7 +149,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v2
         with:
-          node-version: 14.x
+          node-version: 16
           cache: yarn
       - name: install
         run: yarn

package.json

@@ -60,6 +60,9 @@
     "projects": [
       {
         "displayName": "test",
+        "moduleNameMapper": {
+          "@eslint/eslintrc/universal": "@eslint/eslintrc/dist/eslintrc-universal.cjs"
+        },
         "testPathIgnorePatterns": [
           "<rootDir>/lib/.*",
           "<rootDir>/src/rules/__tests__/fixtures/*",
@@ -100,14 +103,14 @@
     "babel-jest": "^27.0.0",
     "babel-plugin-replace-ts-export-assignment": "^0.0.2",
     "dedent": "^0.7.0",
-    "eslint": "^6.0.0 || ^7.0.0",
+    "eslint": "^6.0.0 || ^7.0.0 || ^8.0.0",
     "eslint-config-prettier": "^8.3.0",
-    "eslint-plugin-eslint-comments": "^3.1.2",
+    "eslint-plugin-eslint-comments": "^3.2.0",
     "eslint-plugin-eslint-config": "^2.0.0",
-    "eslint-plugin-eslint-plugin": "^3.5.3",
-    "eslint-plugin-import": "^2.25.1",
-    "eslint-plugin-node": "^11.0.0",
-    "eslint-plugin-prettier": "^3.4.1",
+    "eslint-plugin-eslint-plugin": "^4.0.0-0",
+    "eslint-plugin-import": "^2.25.2",
+    "eslint-plugin-node": "^11.1.0",
+    "eslint-plugin-prettier": "^4.0.0",
     "husky": "^7.0.2",
     "is-ci": "^3.0.0",
     "jest": "^27.0.0",
@@ -123,7 +126,7 @@
   },
   "peerDependencies": {
     "@typescript-eslint/eslint-plugin": "^4.0.0 || ^5.0.0",
-    "eslint": "^6.0.0 || ^7.0.0"
+    "eslint": "^6.0.0 || ^7.0.0 || ^8.0.0"
   },
   "peerDependenciesMeta": {
     "@typescript-eslint/eslint-plugin": {
@@ -149,5 +152,8 @@
       "@semantic-release/git",
       "@semantic-release/github"
     ]
+  },
+  "resolutions": {
+    "@typescript-eslint/experimental-utils": "^5.0.0"
   }
 }

src/index.ts

@@ -8,13 +8,6 @@ type RuleModule = TSESLint.RuleModule<string, unknown[]> & {
   meta: Required<Pick<TSESLint.RuleMetaData<string>, 'docs'>>;
 };
 
-// v5 of `@typescript-eslint/experimental-utils` removed this
-declare module '@typescript-eslint/experimental-utils/dist/ts-eslint/Rule' {
-  export interface RuleMetaDataDocs {
-    category: 'Best Practices' | 'Possible Errors';
-  }
-}
-
 // copied from https://github.com/babel/babel/blob/d8da63c929f2d28c401571e2a43166678c555bc4/packages/babel-helpers/src/helpers.js#L602-L606
 /* istanbul ignore next */
 const interopRequireDefault = (obj: any): { default: any } =>

src/rules/__tests__/no-jasmine-globals.test.ts

@@ -21,6 +21,7 @@ ruleTester.run('no-jasmine-globals', rule, {
   invalid: [
     {
       code: 'spyOn(some, "object")',
+      output: null,
       errors: [
         {
           messageId: 'illegalGlobal',
@@ -32,6 +33,7 @@ ruleTester.run('no-jasmine-globals', rule, {
     },
     {
       code: 'spyOnProperty(some, "object")',
+      output: null,
       errors: [
         {
           messageId: 'illegalGlobal',
@@ -43,10 +45,12 @@ ruleTester.run('no-jasmine-globals', rule, {
     },
     {
       code: 'fail()',
+      output: null,
       errors: [{ messageId: 'illegalFail', column: 1, line: 1 }],
     },
     {
       code: 'pending()',
+      output: null,
       errors: [{ messageId: 'illegalPending', column: 1, line: 1 }],
     },
     {
@@ -56,10 +60,12 @@ ruleTester.run('no-jasmine-globals', rule, {
     },
     {
       code: 'jasmine.DEFAULT_TIMEOUT_INTERVAL = function() {}',
+      output: null,
       errors: [{ messageId: 'illegalJasmine', column: 1, line: 1 }],
     },
     {
       code: 'jasmine.addMatchers(matchers)',
+      output: null,
       errors: [
         {
           messageId: 'illegalMethod',
@@ -71,6 +77,7 @@ ruleTester.run('no-jasmine-globals', rule, {
     },
     {
       code: 'jasmine.createSpy()',
+      output: null,
       errors: [
         {
           messageId: 'illegalMethod',
@@ -151,30 +158,37 @@ ruleTester.run('no-jasmine-globals', rule, {
     },
     {
       code: 'jasmine.getEnv()',
+      output: null,
       errors: [{ messageId: 'illegalJasmine', column: 1, line: 1 }],
     },
     {
       code: 'jasmine.empty()',
+      output: null,
       errors: [{ messageId: 'illegalJasmine', column: 1, line: 1 }],
     },
     {
       code: 'jasmine.falsy()',
+      output: null,
       errors: [{ messageId: 'illegalJasmine', column: 1, line: 1 }],
     },
     {
       code: 'jasmine.truthy()',
+      output: null,
       errors: [{ messageId: 'illegalJasmine', column: 1, line: 1 }],
     },
     {
       code: 'jasmine.arrayWithExactContents()',
+      output: null,
       errors: [{ messageId: 'illegalJasmine', column: 1, line: 1 }],
     },
     {
       code: 'jasmine.clock()',
+      output: null,
       errors: [{ messageId: 'illegalJasmine', column: 1, line: 1 }],
     },
     {
       code: 'jasmine.MAX_PRETTY_PRINT_ARRAY_LENGTH = 42',
+      output: null,
       errors: [{ messageId: 'illegalJasmine', column: 1, line: 1 }],
     },
   ],

src/rules/__tests__/utils.test.ts

@@ -18,7 +18,6 @@ const rule = createRule({
   name: __filename,
   meta: {
     docs: {
-      category: 'Possible Errors',
       description: 'Fake rule for testing AST guards',
       recommended: false,
     },

src/rules/consistent-test-it.ts

@@ -39,7 +39,6 @@ export default createRule<
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
       description: 'Have control over `test` and `it` usages',
       recommended: false,
     },

src/rules/expect-expect.ts

@@ -47,7 +47,6 @@ export default createRule<
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
       description: 'Enforce assertion to be made in a test body',
       recommended: 'warn',
     },

src/rules/max-nested-describe.ts

@@ -8,7 +8,6 @@ export default createRule({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
       description: 'Enforces a maximum depth to nested describe calls',
       recommended: false,
     },

src/rules/no-alias-methods.ts

@@ -4,7 +4,6 @@ export default createRule({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
       description: 'Disallow alias methods',
       recommended: false,
     },

src/rules/no-commented-out-tests.ts

@@ -11,7 +11,6 @@ export default createRule({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
       description: 'Disallow commented out tests',
       recommended: 'warn',
     },

src/rules/no-conditional-expect.ts

@@ -22,7 +22,6 @@ export default createRule({
   meta: {
     docs: {
       description: 'Prevent calling `expect` conditionally',
-      category: 'Best Practices',
       recommended: 'error',
     },
     messages: {

src/rules/no-deprecated-functions.ts

@@ -27,7 +27,6 @@ export default createRule({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
       description: 'Disallow use of deprecated functions',
       recommended: 'error',
     },

src/rules/no-disabled-tests.ts

@@ -4,7 +4,6 @@ export default createRule({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
       description: 'Disallow disabled tests',
       recommended: 'warn',
     },

src/rules/no-done-callback.ts

@@ -33,7 +33,6 @@ export default createRule({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
       description: 'Avoid using a callback in asynchronous tests and hooks',
       recommended: 'error',
       suggestion: true,

src/rules/no-duplicate-hooks.ts

@@ -11,7 +11,6 @@ export default createRule({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
       description: 'Disallow duplicate setup and teardown hooks',
       recommended: false,
     },

src/rules/no-export.ts

@@ -8,7 +8,6 @@ export default createRule({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
       description: 'Disallow using `exports` in files containing tests',
       recommended: 'error',
     },

src/rules/no-focused-tests.ts

@@ -38,7 +38,6 @@ export default createRule({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
       description: 'Disallow focused tests',
       recommended: 'error',
       suggestion: true,

src/rules/no-hooks.ts

@@ -7,7 +7,6 @@ export default createRule<
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
       description: 'Disallow setup and teardown hooks',
       recommended: false,
     },

src/rules/no-identical-title.ts

@@ -21,7 +21,6 @@ export default createRule({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
       description: 'Disallow identical titles',
       recommended: 'error',
     },

src/rules/no-if.ts

@@ -41,7 +41,6 @@ export default createRule({
   meta: {
     docs: {
       description: 'Disallow conditional logic',
-      category: 'Best Practices',
       recommended: false,
     },
     messages: {

src/rules/no-interpolation-in-snapshots.ts

@@ -5,7 +5,6 @@ export default createRule({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
       description: 'Disallow string interpolation inside snapshots',
       recommended: 'error',
     },

src/rules/no-jasmine-globals.ts

@@ -10,7 +10,6 @@ export default createRule({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
       description: 'Disallow Jasmine globals',
       recommended: 'error',
     },

src/rules/no-jest-import.ts

@@ -7,7 +7,6 @@ export default createRule({
     type: 'problem',
     docs: {
       description: 'Disallow importing Jest',
-      category: 'Best Practices',
       recommended: 'error',
     },
     messages: {