Merge pull request #3057 from D4N14L/user/danade/eslint8

[eslint-plugin] Support ESLint 8

Author: iclanton

apps/heft/package.json

@@ -53,7 +53,7 @@
     "@rushstack/heft": "0.42.3",
     "@rushstack/heft-node-rig": "1.2.32",
     "@types/argparse": "1.0.38",
-    "@types/eslint": "7.2.0",
+    "@types/eslint": "8.2.0",
     "@types/glob": "7.1.1",
     "@types/heft-jest": "1.0.1",
     "@types/node": "12.20.24",

apps/heft/src/plugins/TypeScriptPlugin/Eslint.ts

@@ -52,7 +52,7 @@ export class Eslint extends LinterBase<TEslint.ESLint.LintResult> {
         'Heft requires ESLint 7 or newer.  Your ESLint version is too old:\n' + this._eslintPackagePath
       );
     }
-    if (majorVersion > 7) {
+    if (majorVersion > 8) {
       // We don't use writeWarningLine() here because, if the person wants to take their chances with
       // a newer ESLint release, their build should be allowed to succeed.
       this._terminal.writeLine(

apps/rush-lib/src/logic/taskRunner/BaseBuilder.ts

@@ -24,21 +24,21 @@ export abstract class BaseBuilder {
   /**
    * Name of the task definition.
    */
-  abstract readonly name: string;
+  public abstract readonly name: string;
 
   /**
    * This flag determines if the task is allowed to be skipped if up to date.
    */
-  abstract isSkipAllowed: boolean;
+  public abstract isSkipAllowed: boolean;
 
   /**
    * Assigned by execute().  True if the build script was an empty string.  Operationally an empty string is
    * like a shell command that succeeds instantly, but e.g. it would be odd to report build time statistics for it.
    */
-  abstract hadEmptyScript: boolean;
+  public abstract hadEmptyScript: boolean;
 
   /**
    * Method to be executed for the task.
    */
-  abstract executeAsync(context: IBuilderContext): Promise<TaskStatus>;
+  public abstract executeAsync(context: IBuilderContext): Promise<TaskStatus>;
 }

build-tests-samples/heft-node-basic-tutorial/package.json

@@ -14,7 +14,7 @@
     "@rushstack/heft-jest-plugin": "workspace:*",
     "@types/heft-jest": "1.0.1",
     "@types/node": "12.20.24",
-    "eslint": "~7.30.0",
+    "eslint": "~8.3.0",
     "typescript": "~4.4.2"
   }
 }

build-tests-samples/heft-node-jest-tutorial/package.json

@@ -13,7 +13,7 @@
     "@rushstack/heft-jest-plugin": "workspace:*",
     "@types/heft-jest": "1.0.1",
     "@types/node": "12.20.24",
-    "eslint": "~7.30.0",
+    "eslint": "~8.3.0",
     "typescript": "~4.4.2"
   }
 }

build-tests-samples/heft-storybook-react-tutorial/package.json

@@ -18,7 +18,7 @@
     "@types/react": "16.9.45",
     "@types/webpack-env": "1.13.0",
     "css-loader": "~4.2.1",
-    "eslint": "~7.30.0",
+    "eslint": "~8.3.0",
     "heft-storybook-react-tutorial-storykit": "workspace:*",
     "html-webpack-plugin": "~5.5.0",
     "react-dom": "~16.13.1",

build-tests-samples/heft-webpack-basic-tutorial/package.json

@@ -17,7 +17,7 @@
     "@types/react-dom": "16.9.8",
     "@types/webpack-env": "1.13.0",
     "css-loader": "~4.2.1",
-    "eslint": "~7.30.0",
+    "eslint": "~8.3.0",
     "html-webpack-plugin": "~5.5.0",
     "react": "~16.13.1",
     "react-dom": "~16.13.1",

build-tests-samples/packlets-tutorial/package.json

@@ -12,7 +12,7 @@
     "@rushstack/eslint-config": "workspace:*",
     "@rushstack/heft": "workspace:*",
     "@types/node": "12.20.24",
-    "eslint": "~7.30.0",
+    "eslint": "~8.3.0",
     "typescript": "~4.4.2"
   }
 }

build-tests/eslint-7-test/.eslintrc.js

@@ -0,0 +1,25 @@
+// This is a workaround for https://github.com/eslint/eslint/issues/3458
+require('@rushstack/eslint-config/patch/modern-module-resolution');
+
+module.exports = {
+  extends: [
+    '@rushstack/eslint-config/profile/node-trusted-tool',
+    '@rushstack/eslint-config/mixins/friendly-locals'
+  ],
+  parserOptions: { tsconfigRootDir: __dirname },
+
+  overrides: [
+    /**
+     * Override the parser from @rushstack/eslint-config. Since the config is coming
+     * from the workspace instead of the external NPM package, the versions of ESLint
+     * and TypeScript that the config consumes will be resolved from the devDependencies
+     * of the config instead of from the eslint-7-test package. Overriding the parser
+     * ensures that the these dependencies come from the eslint-7-test package. See:
+     * https://github.com/microsoft/rushstack/issues/3021
+     */
+    {
+      files: ['*.ts', '*.tsx'],
+      parser: '@typescript-eslint/parser'
+    }
+  ]
+};

build-tests/eslint-7-test/README.md

@@ -0,0 +1,6 @@
+# eslint-7-test
+
+This project folder is one of the **build-tests** for the Rushstack [ESLint configuration](https://www.npmjs.com/package/@rushstack/eslint-config) (and by extension, the [ESLint plugin](https://www.npmjs.com/package/@rushstack/eslint-plugin))
+package. This project builds using ESLint v7 and contains a simple index file to ensure that the build runs ESLint successfully against source code.
+
+Please see the [ESLint Heft task documentation](https://rushstack.io/pages/heft_tasks/eslint/) for documentation and tutorials.