feat(linter): remove output generation in linter

nrwl · Dec 5, 2023 · 709e54c · 709e54c
1 parent b55f820
commit 709e54c
Show file tree

Hide file tree

Showing 28 changed files with 12 additions and 134 deletions.
diff --git a/graph/client/src/assets/release-static/environment.js b/graph/client/src/assets/release-static/environment.js
@@ -139,7 +139,6 @@ window.projectGraphResponse = {
                 'libs/products/product-detail-page/src/**/*.html',
               ],
             },
-            outputs: ['{options.outputFile}'],
             inputs: ['default', '{workspaceRoot}/.eslintrc.json'],
           },
           test: {
@@ -261,7 +260,6 @@ window.projectGraphResponse = {
                 'libs/shared/product/state/src/**/*.html',
               ],
             },
-            outputs: ['{options.outputFile}'],
             inputs: ['default', '{workspaceRoot}/.eslintrc.json'],
           },
           test: {
@@ -329,7 +327,6 @@ window.projectGraphResponse = {
                 'libs/shared/product/types/src/**/*.html',
               ],
             },
-            outputs: ['{options.outputFile}'],
             inputs: ['default', '{workspaceRoot}/.eslintrc.json'],
           },
         },
@@ -393,7 +390,6 @@ window.projectGraphResponse = {
                 'libs/shared/product/data/src/**/*.html',
               ],
             },
-            outputs: ['{options.outputFile}'],
             inputs: ['default', '{workspaceRoot}/.eslintrc.json'],
           },
         },
@@ -498,7 +494,6 @@ window.projectGraphResponse = {
                 'libs/products/home-page/src/**/*.html',
               ],
             },
-            outputs: ['{options.outputFile}'],
             inputs: ['default', '{workspaceRoot}/.eslintrc.json'],
           },
           test: {
@@ -616,7 +611,6 @@ window.projectGraphResponse = {
                 'libs/shared/cart/state/src/**/*.html',
               ],
             },
-            outputs: ['{options.outputFile}'],
             inputs: ['default', '{workspaceRoot}/.eslintrc.json'],
           },
           test: {
@@ -702,7 +696,6 @@ window.projectGraphResponse = {
                 'libs/shared/product/ui/src/**/*.html',
               ],
             },
-            outputs: ['{options.outputFile}'],
             inputs: ['default', '{workspaceRoot}/.eslintrc.json'],
           },
           test: {
@@ -766,7 +759,6 @@ window.projectGraphResponse = {
                 'libs/shared/e2e-utils/src/**/*.html',
               ],
             },
-            outputs: ['{options.outputFile}'],
             inputs: ['default', '{workspaceRoot}/.eslintrc.json'],
           },
         },
@@ -846,7 +838,6 @@ window.projectGraphResponse = {
                 'libs/cart/cart-page/src/**/*.html',
               ],
             },
-            outputs: ['{options.outputFile}'],
             inputs: ['default', '{workspaceRoot}/.eslintrc.json'],
           },
           test: {
@@ -987,7 +978,6 @@ window.projectGraphResponse = {
                 'libs/shared/header/src/**/*.html',
               ],
             },
-            outputs: ['{options.outputFile}'],
             inputs: ['default', '{workspaceRoot}/.eslintrc.json'],
           },
           test: {
@@ -1055,7 +1045,6 @@ window.projectGraphResponse = {
                 'libs/shared/jsxify/src/**/*.html',
               ],
             },
-            outputs: ['{options.outputFile}'],
             inputs: ['default', '{workspaceRoot}/.eslintrc.json'],
           },
         },
@@ -1154,7 +1143,6 @@ window.projectGraphResponse = {
             options: {
               lintFilePatterns: ['apps/products-e2e/**/*.{js,ts}'],
             },
-            outputs: ['{options.outputFile}'],
             inputs: ['default', '{workspaceRoot}/.eslintrc.json'],
           },
         },
@@ -1226,7 +1214,6 @@ window.projectGraphResponse = {
             options: {
               lintFilePatterns: ['apps/cart-e2e/**/*.{ts,tsx,js,jsx}'],
             },
-            outputs: ['{options.outputFile}'],
             inputs: ['default', '{workspaceRoot}/.eslintrc.json'],
           },
         },
@@ -1432,7 +1419,6 @@ window.projectGraphResponse = {
                 'apps/products/src/**/*.html',
               ],
             },
-            outputs: ['{options.outputFile}'],
             inputs: ['default', '{workspaceRoot}/.eslintrc.json'],
           },
           test: {
@@ -1637,7 +1623,6 @@ window.projectGraphResponse = {
             options: {
               lintFilePatterns: ['apps/cart/**/*.{ts,tsx,js,jsx}'],
             },
-            outputs: ['{options.outputFile}'],
             inputs: ['default', '{workspaceRoot}/.eslintrc.json'],
           },
           test: {

diff --git a/packages/angular/src/generators/add-linting/add-linting.spec.ts b/packages/angular/src/generators/add-linting/add-linting.spec.ts
@@ -72,7 +72,6 @@ describe('addLinting generator', () => {
     const project = readProjectConfiguration(tree, appProjectName);
     expect(project.targets.lint).toEqual({
       executor: '@nx/eslint:lint',
-      outputs: ['{options.outputFile}'],
     });
   });
 });
diff --git a/packages/angular/src/generators/application/__snapshots__/application.spec.ts.snap b/packages/angular/src/generators/application/__snapshots__/application.spec.ts.snap
@@ -267,9 +267,6 @@ exports[`app --project-name-and-root-format=derived should generate correctly wh
     },
     "lint": {
       "executor": "@nx/eslint:lint",
-      "outputs": [
-        "{options.outputFile}",
-      ],
     },
     "serve": {
       "configurations": {
@@ -333,9 +330,6 @@ exports[`app --project-name-and-root-format=derived should generate correctly wh
     },
     "lint": {
       "executor": "@nx/eslint:lint",
-      "outputs": [
-        "{options.outputFile}",
-      ],
     },
   },
 }
@@ -492,9 +486,6 @@ exports[`app --project-name-and-root-format=derived should generate correctly wh
     },
     "lint": {
       "executor": "@nx/eslint:lint",
-      "outputs": [
-        "{options.outputFile}",
-      ],
     },
     "serve": {
       "configurations": {
@@ -558,9 +549,6 @@ exports[`app --project-name-and-root-format=derived should generate correctly wh
     },
     "lint": {
       "executor": "@nx/eslint:lint",
-      "outputs": [
-        "{options.outputFile}",
-      ],
     },
   },
 }
@@ -941,9 +929,6 @@ exports[`app nested should create project configs 1`] = `
     },
     "lint": {
       "executor": "@nx/eslint:lint",
-      "outputs": [
-        "{options.outputFile}",
-      ],
     },
     "serve": {
       "configurations": {
@@ -1007,9 +992,6 @@ exports[`app nested should create project configs 2`] = `
     },
     "lint": {
       "executor": "@nx/eslint:lint",
-      "outputs": [
-        "{options.outputFile}",
-      ],
     },
   },
 }
@@ -1079,9 +1061,6 @@ exports[`app not nested should create project configs 1`] = `
     },
     "lint": {
       "executor": "@nx/eslint:lint",
-      "outputs": [
-        "{options.outputFile}",
-      ],
     },
     "serve": {
       "configurations": {
@@ -1145,9 +1124,6 @@ exports[`app not nested should create project configs 2`] = `
     },
     "lint": {
       "executor": "@nx/eslint:lint",
-      "outputs": [
-        "{options.outputFile}",
-      ],
     },
   },
 }

diff --git a/packages/angular/src/generators/application/application.spec.ts b/packages/angular/src/generators/application/application.spec.ts
@@ -506,18 +506,12 @@ describe('app', () => {
           .toMatchInlineSnapshot(`
           {
             "executor": "@nx/eslint:lint",
-            "outputs": [
-              "{options.outputFile}",
-            ],
           }
         `);
         expect(readProjectConfiguration(appTree, 'my-app-e2e').targets.lint)
           .toMatchInlineSnapshot(`
           {
             "executor": "@nx/eslint:lint",
-            "outputs": [
-              "{options.outputFile}",
-            ],
           }
         `);
       });

diff --git a/packages/angular/src/generators/library/library.spec.ts b/packages/angular/src/generators/library/library.spec.ts
@@ -1129,9 +1129,6 @@ describe('lib', () => {
           .toMatchInlineSnapshot(`
           {
             "executor": "@nx/eslint:lint",
-            "outputs": [
-              "{options.outputFile}",
-            ],
           }
         `);
       });

diff --git a/packages/angular/src/generators/ng-add/__snapshots__/migrate-from-angular-cli.spec.ts.snap b/packages/angular/src/generators/ng-add/__snapshots__/migrate-from-angular-cli.spec.ts.snap
@@ -100,6 +100,15 @@ exports[`workspace move to nx layout should create nx.json 1`] = `
     "sharedGlobals": [],
   },
   "targetDefaults": {
+    "@nx/eslint:lint": {
+      "cache": true,
+      "inputs": [
+        "default",
+        "{workspaceRoot}/.eslintrc.json",
+        "{workspaceRoot}/.eslintignore",
+        "{workspaceRoot}/eslint.config.js",
+      ],
+    },
     "build": {
       "cache": true,
       "dependsOn": [

diff --git a/packages/cypress/src/generators/cypress-project/__snapshots__/cypress-project.spec.ts.snap b/packages/cypress/src/generators/cypress-project/__snapshots__/cypress-project.spec.ts.snap
@@ -50,9 +50,6 @@ exports[`Cypress Project < v7 nested should update configuration 1`] = `
   },
   "lint": {
     "executor": "@nx/eslint:lint",
-    "outputs": [
-      "{options.outputFile}",
-    ],
   },
 }
 `;
@@ -91,9 +88,6 @@ exports[`Cypress Project < v7 project with directory in its name should update c
   },
   "lint": {
     "executor": "@nx/eslint:lint",
-    "outputs": [
-      "{options.outputFile}",
-    ],
   },
 }
 `;
@@ -111,9 +105,6 @@ exports[`Cypress Project < v7 should update project configuration (baseUrl) 1`]
   },
   "lint": {
     "executor": "@nx/eslint:lint",
-    "outputs": [
-      "{options.outputFile}",
-    ],
   },
 }
 `;
@@ -136,9 +127,6 @@ exports[`Cypress Project < v7 should update project configuration 1`] = `
   },
   "lint": {
     "executor": "@nx/eslint:lint",
-    "outputs": [
-      "{options.outputFile}",
-    ],
   },
 }
 `;
@@ -161,9 +149,6 @@ exports[`Cypress Project < v7 should update target configurations 1`] = `
   },
   "lint": {
     "executor": "@nx/eslint:lint",
-    "outputs": [
-      "{options.outputFile}",
-    ],
   },
 }
 `;
@@ -243,8 +228,5 @@ exports[`Cypress Project > v10 should set right path names in \`tsconfig.e2e.jso
 exports[`Cypress Project > v10 should update configuration when eslint is passed 1`] = `
 {
   "executor": "@nx/eslint:lint",
-  "outputs": [
-    "{options.outputFile}",
-  ],
 }
 `;
diff --git a/...ess/src/generators/migrate-to-cypress-11/__snapshots__/migrate-to-cypress-11.spec.ts.snap b/...ess/src/generators/migrate-to-cypress-11/__snapshots__/migrate-to-cypress-11.spec.ts.snap
@@ -182,9 +182,6 @@ exports[`convertToCypressTen convertCypressProject should infer targets with --a
   },
   "lint": {
     "executor": "@nx/eslint:lint",
-    "outputs": [
-      "{options.outputFile}",
-    ],
   },
 }
 `;
@@ -242,9 +239,6 @@ exports[`convertToCypressTen convertCypressProject should not break when an inva
   },
   "lint": {
     "executor": "@nx/eslint:lint",
-    "outputs": [
-      "{options.outputFile}",
-    ],
   },
 }
 `;

diff --git a/packages/eslint/migrations.json b/packages/eslint/migrations.json
@@ -39,7 +39,7 @@
       "implementation": "./src/migrations/update-17-2-0/simplify-eslint-patterns"
     },
     "move-options-to-target-defaults": {
-      "version": "17.2.0-beta.0",
+      "version": "17.2.0-beta.13",
       "description": "Move executor options to target defaults",
       "implementation": "./src/migrations/update-17-2-0/move-options-to-target-defaults"
     }

diff --git a/packages/eslint/src/generators/lint-project/lint-project.spec.ts b/packages/eslint/src/generators/lint-project/lint-project.spec.ts
@@ -75,9 +75,6 @@ describe('@nx/eslint:lint-project', () => {
     expect(projectConfig.targets.lint).toMatchInlineSnapshot(`
       {
         "executor": "@nx/eslint:lint",
-        "outputs": [
-          "{options.outputFile}",
-        ],
       }
     `);
   });
@@ -100,9 +97,6 @@ describe('@nx/eslint:lint-project', () => {
             "libs/test-lib/src/**/*.ts",
           ],
         },
-        "outputs": [
-          "{options.outputFile}",
-        ],
       }
     `);
   });
@@ -149,9 +143,6 @@ describe('@nx/eslint:lint-project', () => {
     expect(projectConfig.targets.lint).toMatchInlineSnapshot(`
       {
         "executor": "@nx/eslint:lint",
-        "outputs": [
-          "{options.outputFile}",
-        ],
       }
     `);
   });
@@ -175,9 +166,6 @@ describe('@nx/eslint:lint-project', () => {
             "{projectRoot}/package.json",
           ],
         },
-        "outputs": [
-          "{options.outputFile}",
-        ],
       }
     `);
   });