build: setup prettier in ng-dev format tool

Sets up prettier in the `yarn ng-dev format` command so
that we can format the code consistently.

Author: devversion

.ng-dev/format.ts

@@ -6,4 +6,5 @@ import {FormatConfig} from '@angular/dev-infra-private/ng-dev/format/config';
  */
 export const format: FormatConfig = {
   buildifier: true,
+  prettier: true,
 };

.prettierignore

@@ -0,0 +1,10 @@
+# These schematic ng-generate files are template files that cannot be formatted.
+src/material/schematics/ng-generate/*/files/**/*
+src/cdk/schematics/ng-generate/*/files/**/*
+
+# Schematic ng-update test cases should not be formatted as that could break the
+# expected output tests. Generated code does is not formatted according to prettier.
+src/cdk/schematics/ng-update/test-cases/**/*_input.ts
+src/cdk/schematics/ng-update/test-cases/**/*_expected_output.ts
+src/material/schematics/ng-update/test-cases/**/*_input.ts
+src/material/schematics/ng-update/test-cases/**/*_expected_output.ts

.prettierrc

@@ -0,0 +1,9 @@
+{
+  "printWidth": 100,
+  "singleQuote": true,
+  "trailingComma": "all",
+  "quoteProps": "preserve",
+  "bracketSpacing": false,
+  "arrowParens": "avoid",
+  "embeddedLanguageFormatting": "off",
+}

scripts/check-mdc-tests.ts

@@ -90,15 +90,18 @@ function getTestNames(files: string[]): string[] {
         // Note that this is a little naive since it'll take the literal text of the test
         // name expression which could include things like string concatenation. It's fine
         // for the limited use cases of the script.
-        testNames.push(node.arguments[0].getText(sourceFile)
-          // Replace the quotes around the test name.
-          .replace(/^['`]|['`]$/g, '')
-          // Strip newlines followed by indentation.
-          .replace(/\n\s+/g, ' ')
-          // Strip escape characters.
-          .replace(/\\/g, '')
-          // Collapse concatenated strings.
-          .replace(/['`]\s+\+\s+['`]/g, ''));
+        testNames.push(
+          node.arguments[0]
+            .getText(sourceFile)
+            // Replace the quotes around the test name.
+            .replace(/^['"`]|['"`]$/g, '')
+            // Strip newlines followed by indentation.
+            .replace(/\n\s+/g, ' ')
+            // Strip escape characters.
+            .replace(/\\/g, '')
+            // Collapse concatenated strings.
+            .replace(/['"`]\s+\+\s+['"`]/g, ''),
+        );
       } else {
         node.forEachChild(walk);
       }

tools/tslint-rules/noImportExportSpacingRule.ts

@@ -15,13 +15,6 @@
         "Use 'readonly T[]' instead."
       ]
     ],
-    "max-line-length": [true, {
-      "limit": 100,
-      "check-strings": true,
-      "check-regex": true,
-      "ignore-pattern": "//\\s+https?://"
-      }
-    ],
     // Disable this flag because of SHA tslint#48b0c597f9257712c7d1f04b55ed0aa60e333f6a
     // TSLint now shows warnings if types for properties are inferred. This rule needs to be
     // disabled because all properties need to have explicit types set to work for Dgeni.
@@ -30,10 +23,6 @@
       true,
       "check-space"
     ],
-    "indent": [
-      true,
-      "spaces"
-    ],
     "eofline": true,
     "no-construct": true,
     "prefer-literal": [true, "object"],
@@ -49,46 +38,13 @@
     "no-var-keyword": true,
     "member-access": [true, "no-public"],
     "no-debugger": true,
-    "one-line": [
-      true,
-      "check-catch",
-      "check-else",
-      "check-open-brace",
-      "check-whitespace"
-    ],
-    "quotemark": [
-      true,
-      "single",
-      "avoid-escape"
-    ],
-    "semicolon": true,
-    "typedef-whitespace": [
-      true,
-      {
-        "call-signature": "nospace",
-        "index-signature": "nospace",
-        "parameter": "nospace",
-        "property-declaration": "nospace",
-        "variable-declaration": "nospace"
-      }
-    ],
-    "curly": true,
     "variable-name": [
       true,
       "ban-keywords",
       "check-format",
       "allow-leading-underscore",
       "allow-snake-case"
     ],
-    "whitespace": [
-      true,
-      "check-branch",
-      "check-decl",
-      "check-operator",
-      "check-separator",
-      "check-type",
-      "check-preblock"
-    ],
     // Bans jasmine helper functions that will prevent the CI from properly running tests.
     "ban": [
       true,
@@ -127,7 +83,6 @@
     // Custom Rules
     "ts-loader": true,
     "no-exposed-todo": true,
-    "no-import-export-spacing": true,
     "no-private-getters": [true, "^_"],
     "no-undecorated-base-class-di": true,
     "no-undecorated-class-with-angular-features": true,