fix(angular): add --skip-format for convert-tslint-to-eslint (#6387)

Author: FrozenPandaz

docs/angular/api-angular/generators/convert-tslint-to-eslint.md

@@ -65,3 +65,11 @@ Default: `true`
 Type: `boolean`
 
 If this conversion leaves no more TSLint usage in the workspace, it will remove TSLint and related dependencies and configuration.
+
+### skipFormat
+
+Default: `false`
+
+Type: `boolean`
+
+Skip formatting files.

docs/angular/api-nest/generators/convert-tslint-to-eslint.md

@@ -53,3 +53,11 @@ Default: `true`
 Type: `boolean`
 
 If this conversion leaves no more TSLint usage in the workspace, it will remove TSLint and related dependencies and configuration.
+
+### skipFormat
+
+Default: `false`
+
+Type: `boolean`
+
+Skip formatting files.

docs/node/api-angular/generators/convert-tslint-to-eslint.md

@@ -65,3 +65,11 @@ Default: `true`
 Type: `boolean`
 
 If this conversion leaves no more TSLint usage in the workspace, it will remove TSLint and related dependencies and configuration.
+
+### skipFormat
+
+Default: `false`
+
+Type: `boolean`
+
+Skip formatting files.

docs/node/api-nest/generators/convert-tslint-to-eslint.md

@@ -53,3 +53,11 @@ Default: `true`
 Type: `boolean`
 
 If this conversion leaves no more TSLint usage in the workspace, it will remove TSLint and related dependencies and configuration.
+
+### skipFormat
+
+Default: `false`
+
+Type: `boolean`
+
+Skip formatting files.

docs/react/api-angular/generators/convert-tslint-to-eslint.md

@@ -65,3 +65,11 @@ Default: `true`
 Type: `boolean`
 
 If this conversion leaves no more TSLint usage in the workspace, it will remove TSLint and related dependencies and configuration.
+
+### skipFormat
+
+Default: `false`
+
+Type: `boolean`
+
+Skip formatting files.

docs/react/api-nest/generators/convert-tslint-to-eslint.md

@@ -53,3 +53,11 @@ Default: `true`
 Type: `boolean`
 
 If this conversion leaves no more TSLint usage in the workspace, it will remove TSLint and related dependencies and configuration.
+
+### skipFormat
+
+Default: `false`
+
+Type: `boolean`
+
+Skip formatting files.

packages/angular/src/generators/add-linting/add-linting.ts

@@ -14,7 +14,10 @@ export async function addLintingGenerator(
   addAngularEsLintDependencies(tree);
   createEsLintConfiguration(tree, options);
   addProjectLintTarget(tree, options);
-  await formatFiles(tree);
+
+  if (!options.skipFormat) {
+    await formatFiles(tree);
+  }
 
   return installTask;
 }

packages/angular/src/generators/add-linting/schema.d.ts

@@ -3,4 +3,5 @@ export interface AddLintingGeneratorSchema {
   projectRoot: string;
   prefix: string;
   setParserOptionsProject?: boolean;
+  skipFormat?: boolean;
 }

packages/angular/src/generators/add-linting/schema.json

@@ -22,6 +22,11 @@
       "type": "boolean",
       "description": "Whether or not to configure the ESLint `parserOptions.project` option. We do not do this by default for lint performance reasons.",
       "default": false
+    },
+    "skipFormat": {
+      "type": "boolean",
+      "description": "Skip formatting files.",
+      "default": false
     }
   },
   "additionalProperties": false,

packages/angular/src/generators/convert-tslint-to-eslint/convert-tslint-to-eslint.ts

@@ -39,6 +39,7 @@ export async function conversionGenerator(
          * does and remove the config again if it doesn't, so that it is most efficient.
          */
         setParserOptionsProject: true,
+        skipFormat: options.skipFormat,
       });
     },
   });
@@ -103,6 +104,7 @@ export async function conversionGenerator(
          * step of this parent generator, if applicable
          */
         removeTSLintIfNoMoreTSLintTargets: false,
+        skipFormat: options.skipFormat,
       });
     } catch {
       logger.warn(
@@ -122,7 +124,9 @@ export async function conversionGenerator(
     uninstallTSLintTask = projectConverter.removeTSLintFromWorkspace();
   }
 
-  await formatFiles(host);
+  if (!options.skipFormat) {
+    await formatFiles(host);
+  }
 
   return async () => {
     await eslintInitInstallTask();