feat: eslint-bulk package

Author: kevin-y-ang

build-tests/eslint-bulk-suppressions-test/.eslint-bulk-suppressions.json

@@ -1,31 +1 @@
-// Copyright (c) Microsoft Corporation. All rights reserved. Licensed under the MIT license.
-// See LICENSE in the project root for license information.
-
-// This is a workaround for https://github.com/eslint/eslint/issues/3458
-require('local-node-rig/profiles/default/includes/eslint/patch/modern-module-resolution');
-// This is a workaround for https://github.com/microsoft/rushstack/issues/3021
-require('local-node-rig/profiles/default/includes/eslint/patch/custom-config-package-names');
-require('local-node-rig/profiles/default/includes/eslint/patch/eslint-bulk-suppressions');
-
-module.exports = {
- extends: [
- 'local-node-rig/profiles/default/includes/eslint/profile/node-trusted-tool',
- 'local-node-rig/profiles/default/includes/eslint/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-8-test package. Overriding the parser
-     * ensures that the these dependencies come from the eslint-8-test package. See:
-     * https://github.com/microsoft/rushstack/issues/3021
-     */
- {
- files: ['*.ts', '*.tsx'],
- parser: '@typescript-eslint/parser'
- }
- ]
-};
+module.exports = {};

build-tests/eslint-bulk-suppressions-test/.eslintrc.js

@@ -0,0 +1,14 @@
+{
+ "suppressions": [
+    {
+ "file": "src/index.ts",
+ "scope": ".",
+ "rule": "prefer-const"
+    },
+    {
+ "file": "src/index.ts",
+ "scope": ".exampleFunction",
+ "rule": "@typescript-eslint/ban-types"
+    }
+  ]
+}

build-tests/eslint-bulk-suppressions-test/client/.eslint-bulk-suppressions.json

@@ -0,0 +1,33 @@
+// Copyright (c) Microsoft Corporation. All rights reserved. Licensed under the MIT license.
+// See LICENSE in the project root for license information.
+
+// This is a workaround for https://github.com/eslint/eslint/issues/3458
+require('local-node-rig/profiles/default/includes/eslint/patch/modern-module-resolution');
+// This is a workaround for https://github.com/microsoft/rushstack/issues/3021
+require('local-node-rig/profiles/default/includes/eslint/patch/custom-config-package-names');
+require('local-node-rig/profiles/default/includes/eslint/patch/eslint-bulk-suppressions');
+
+module.exports = {
+ extends: [
+ 'plugin:@typescript-eslint/recommended'
+ // 'local-node-rig/profiles/default/includes/eslint/profile/node-trusted-tool',
+ // 'local-node-rig/profiles/default/includes/eslint/mixins/friendly-locals'
+ ],
+ ignorePatterns: ['.eslintrc.js'],
+
+ 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-8-test package. Overriding the parser
+     * ensures that the these dependencies come from the eslint-8-test package. See:
+     * https://github.com/microsoft/rushstack/issues/3021
+     */
+ {
+ files: ['**/*.ts', '**/*.tsx'],
+ parser: '@typescript-eslint/parser',
+ parserOptions: { project: '../tsconfig.json', tsconfigRootDir: __dirname }
+ }
+ ]
+};

build-tests/eslint-bulk-suppressions-test/client/.eslintrc.js

@@ -0,0 +1,64 @@
+// Copyright (c) Microsoft Corporation. All rights reserved. Licensed under the MIT license.
+// See LICENSE in the project root for license information.
+
+/* Top-level scope code samples */
+// scopeId: '.'
+let exampleString: string = 5 + '';
+
+const exampleObject = {
+ exampleString: exampleString
+};
+
+/* Function scope code samples */
+export function exampleFunction() {
+ const {}: Object = exampleObject;
+
+ // scopeId: '.exampleFunction'
+ !!!exampleString as Boolean;
+}
+
+// scope: '.ArrowFunctionExpression',
+// export const x = () => {},
+//   // scopeId: '.y'
+//   y = () => {},
+//   // scopeId: '.z'
+//   z = () => {};
+
+// /* Class scope code samples */
+// export class ExampleClass {
+//   // scopeId: '.ExampleClass'
+//   exampleClassProperty: String = exampleString + '4';
+
+//   exampleMethod() {
+//     // scopeId: '.exampleClass.exampleMethod'
+//     var exampleVar;
+//     return exampleVar;
+//   }
+// }
+
+// /* Variable and anonymous constructs code samples */
+// export const exampleArrowFunction = () => {
+//   const exampleBoolean = true;
+//   if (exampleBoolean) {
+//   }
+
+//   exampleObject['exampleString'];
+// };
+
+// export const exampleAnonymousClass = class {
+//   exampleClassProperty = 'x' + 'y';
+
+//   // scopeId: '.exampleAnonymousClass.constructor'
+//   constructor() {}
+
+//   set exampleSetGet(val: string) {
+//     // scopeId: '.exampleAnonymousClass.exampleSetGet'
+//     let exampleVariable: Number = 1;
+//     this.exampleClassProperty = val + exampleVariable;
+//   }
+
+//   get exampleSetGet() {
+//     // scopeId: '.exampleAnonymousClass.exampleSetGet'
+//     return this.exampleClassProperty as String as string;
+//   }
+// };

build-tests/eslint-bulk-suppressions-test/client/src/index.ts

@@ -6,7 +6,8 @@
  "scripts": {
  "eslint": "eslint .",
  "eslint-bulk-suppress-all": "ESLINT_BULK_SUPPRESS_RULES=* eslint .",
- "eslint-without-bulk-suppressions": "USE_ESLINT_BULK_SUPPRESSIONS=false eslint ."
+ "eslint-without-bulk-suppressions": "USE_ESLINT_BULK_SUPPRESSIONS=false eslint .",
+ "debug": "CLEANUP_ESLINT_BULK_SUPPRESSIONS=true eslint /Users/bytedance/rushstack/build-tests/eslint-bulk-suppressions-test/server"
   },
  "devDependencies": {
  "@rushstack/heft": "workspace:*",

build-tests/eslint-bulk-suppressions-test/package.json

@@ -0,0 +1,14 @@
+{
+ "suppressions": [
+    {
+ "file": "src/index.ts",
+ "scope": ".exampleObject2.exampleObjectMethod",
+ "rule": "@typescript-eslint/no-inferrable-types"
+    },
+    {
+ "file": "src/index.ts",
+ "scope": ".exampleObject2.exampleObjectProperty",
+ "rule": "@typescript-eslint/no-empty-function"
+    }
+  ]
+}