refactor: re-format all tracked files with prettier

Reformats all tracked files with prettier so that formatting
is consistent across the repo.

Author: devversion

.circleci/rebase-pr.js

@@ -21,7 +21,6 @@
 // tslint:disable:no-console
 const {execSync: execSync_} = require('child_process');
 
-
 /** A regex to select a ref that matches our semver refs. */
 const semverRegex = /^(\d+)\.(\d+)\.x$/;
 
@@ -34,7 +33,9 @@ function exec(command, allowStderr = true) {
   let output = new String();
   output.code = 0;
   try {
-    output += execSync_(command, {stdio: ['pipe', 'pipe', 'pipe']}).toString().trim();
+    output += execSync_(command, {stdio: ['pipe', 'pipe', 'pipe']})
+      .toString()
+      .trim();
   } catch (err) {
     allowStderr && console.error(err.stderr.toString());
     output.code = err.status;
@@ -44,7 +45,6 @@ function exec(command, allowStderr = true) {
 
 // Run
 
-
 // Helpers
 async function _main() {
   const refs = await getRefsAndShasForChange();
@@ -59,12 +59,16 @@ async function _main() {
   console.log();
 
   // Get the count of commits between the latest commit from origin and the common ancestor SHA.
-  const commitCount = exec(`git rev-list --count origin/${refs.base.ref}...${refs.commonAncestorSha}`);
+  const commitCount = exec(
+    `git rev-list --count origin/${refs.base.ref}...${refs.commonAncestorSha}`,
+  );
   console.log(`Checking ${commitCount} commits for changes in the CircleCI config file.`);
 
   // Check if the files changed between the latest commit from origin and the common ancestor SHA
   // includes the CircleCI config.
-  const circleCIConfigChanged = exec(`git diff --name-only origin/${refs.base.ref} ${refs.commonAncestorSha} -- .circleci/config.yml`);
+  const circleCIConfigChanged = exec(
+    `git diff --name-only origin/${refs.base.ref} ${refs.commonAncestorSha} -- .circleci/config.yml`,
+  );
 
   if (!!circleCIConfigChanged) {
     throw Error(`
@@ -90,8 +94,6 @@ async function _main() {
   console.log(`Rebased current branch onto ${refs.base.ref}.`);
 }
 
-
-
 /**
  * Sort a list of fullpath refs into a list and then provide the first entry.
  *
@@ -116,8 +118,11 @@ function getRefFromBranchList(gitOutput) {
     if (aIsSemver && bIsSemver) {
       const [, aMajor, aMinor] = a.match(semverRegex);
       const [, bMajor, bMinor] = b.match(semverRegex);
-      return parseInt(bMajor, 10) - parseInt(aMajor, 10) ||
-          parseInt(aMinor, 10) - parseInt(bMinor, 10) || 0;
+      return (
+        parseInt(bMajor, 10) - parseInt(aMajor, 10) ||
+        parseInt(aMinor, 10) - parseInt(bMinor, 10) ||
+        0
+      );
     }
     if (aIsSemver) {
       return -1;
@@ -168,7 +173,6 @@ function addAndFetchRemote(owner, name) {
   return remoteName;
 }
 
-
 /** Get the ref and latest shas for the provided sha on a specific remote. */
 function getRefAndShas(sha, owner, name) {
   const remoteName = addAndFetchRemote(owner, name);
@@ -181,19 +185,18 @@ function getRefAndShas(sha, owner, name) {
   return {remote: remoteName, ref, latestSha, sha};
 }
 
-
 /** Gets the refs and shas for the base and target of the current environment. */
 function getRefsAndShasForChange() {
   const base = getRefAndShas(
     process.env['CIRCLE_GIT_BASE_REVISION'],
     process.env['CIRCLE_PROJECT_USERNAME'],
-    process.env['CIRCLE_PROJECT_REPONAME']
+    process.env['CIRCLE_PROJECT_REPONAME'],
   );
 
   const target = getRefAndShas(
     process.env['CIRCLE_GIT_REVISION'],
     process.env['CIRCLE_PR_USERNAME'],
-    process.env['CIRCLE_PR_REPONAME']
+    process.env['CIRCLE_PR_REPONAME'],
   );
 
   const commonAncestorSha = getCommonAncestorSha(base.sha, target.sha);
@@ -204,7 +207,6 @@ function getRefsAndShasForChange() {
   };
 }
 
-
 _main().catch(err => {
   console.log('Failed to rebase on top of target branch.\n');
   console.error(err);

.ng-dev/caretaker.ts

@@ -10,6 +10,6 @@ export const caretaker: CaretakerConfig = {
     {
       name: 'Triage Queue',
       query: `is:open label:"needs triage"`,
-    }
-  ]
+    },
+  ],
 };

.ng-dev/commit-message.ts

@@ -113,6 +113,6 @@ export const commitMessage: CommitMessageConfig = {
     'material/toolbar',
     'material/tooltip',
     'material/tree',
-    'youtube-player'
+    'youtube-player',
   ],
 };

.ng-dev/pull-request.ts

@@ -11,9 +11,7 @@ export const pullRequest: PullRequestConfig = {
   // use the Github API merge strategy. That way we ensure that PRs show up as `Merged`.
   githubApiMerge: {
     default: 'squash',
-    labels: [
-      {pattern: 'preserve commits', method: 'rebase'}
-    ]
+    labels: [{pattern: 'preserve commits', method: 'rebase'}],
   },
   mergeReadyLabel: 'merge ready',
   commitMessageFixupLabel: 'commit message fixup',

.ng-dev/release.ts

@@ -4,9 +4,7 @@ import {SemVer} from 'semver';
 import {assertValidNpmPackageOutput} from '../tools/release-checks/npm-package-output';
 import {fork} from 'child_process';
 import {join} from 'path';
-import {
-  FatalReleaseActionError
-} from '@angular/dev-infra-private/ng-dev/release/publish/actions-error';
+import {FatalReleaseActionError} from '@angular/dev-infra-private/ng-dev/release/publish/actions-error';
 
 const actionProto = ReleaseAction.prototype as any;
 const _origStageFn = actionProto.stageVersionForBranchAndCreatePullRequest;
@@ -20,8 +18,9 @@ async function runStagingReleaseChecks(newVersion: SemVer) {
     // directly call into the release checks, the `.ng-dev/release` config would be
     // cached by NodeJS and release checks would potentially check for packages which
     // no longer exist in the publish branch (or the other way around).
-    const releaseChecksProcess = fork(
-      join(__dirname, '../tools/release-checks/index.js'), [newVersion.format()]);
+    const releaseChecksProcess = fork(join(__dirname, '../tools/release-checks/index.js'), [
+      newVersion.format(),
+    ]);
 
     releaseChecksProcess.on('close', code => {
       if (code !== 0) {
@@ -37,16 +36,18 @@ async function runStagingReleaseChecks(newVersion: SemVer) {
 // before staging a release. This is temporary until the dev-infra team has implemented
 // a more generic solution to running sanity checks before releasing (potentially building
 // some of the checks we have in the components repository into the release tool).
-actionProto.stageVersionForBranchAndCreatePullRequest = async function(newVersion: SemVer) {
+actionProto.stageVersionForBranchAndCreatePullRequest = async function (newVersion: SemVer) {
   await runStagingReleaseChecks(newVersion);
 
   return await _origStageFn.apply(this, arguments);
 };
 
 // Patches the `@angular/dev-infra-private` release tool to perform sanity
 // checks of the NPM package output, before publishing to NPM.
-actionProto._verifyPackageVersions =
-    async function(newVersion: SemVer, builtPackages: BuiltPackage[]) {
+actionProto._verifyPackageVersions = async function (
+  newVersion: SemVer,
+  builtPackages: BuiltPackage[],
+) {
   await assertValidNpmPackageOutput(builtPackages, newVersion);
 
   return await _origVerifyFn.apply(this, arguments);

build-config.js

@@ -37,5 +37,5 @@ module.exports = {
   projectDir: __dirname,
   packagesDir: join(__dirname, 'src'),
   outputDir: join(__dirname, 'dist'),
-  licenseBanner: buildLicense
+  licenseBanner: buildLicense,
 };

integration/ng-add/karma.conf.js

@@ -15,7 +15,7 @@ module.exports = function (config) {
       require('karma-chrome-launcher'),
       require('karma-jasmine-html-reporter'),
       require('karma-coverage'),
-      require('@angular-devkit/build-angular/plugins/karma')
+      require('@angular-devkit/build-angular/plugins/karma'),
     ],
     client: {
       jasmine: {
@@ -24,18 +24,15 @@ module.exports = function (config) {
         // for example, you can disable the random execution with `random: false`
         // or set a specific seed with `seed: 4321`
       },
-      clearContext: false // leave Jasmine Spec Runner output visible in browser
+      clearContext: false, // leave Jasmine Spec Runner output visible in browser
     },
     jasmineHtmlReporter: {
-      suppressAll: true // removes the duplicated traces
+      suppressAll: true, // removes the duplicated traces
     },
     coverageReporter: {
       dir: require('path').join(__dirname, './coverage/ng-add'),
       subdir: '.',
-      reporters: [
-        { type: 'html' },
-        { type: 'text-summary' }
-      ]
+      reporters: [{type: 'html'}, {type: 'text-summary'}],
     },
     reporters: ['progress', 'kjhtml'],
     port: 9876,
@@ -44,6 +41,6 @@ module.exports = function (config) {
     autoWatch: true,
     browsers: ['ChromeHeadless'],
     singleRun: false,
-    restartOnFileChange: true
+    restartOnFileChange: true,
   });
 };

integration/ng-add/src/app/app-routing.module.ts

@@ -1,10 +1,10 @@
-import { NgModule } from '@angular/core';
-import { RouterModule, Routes } from '@angular/router';
+import {NgModule} from '@angular/core';
+import {RouterModule, Routes} from '@angular/router';
 
 const routes: Routes = [];
 
 @NgModule({
   imports: [RouterModule.forRoot(routes)],
-  exports: [RouterModule]
+  exports: [RouterModule],
 })
-export class AppRoutingModule { }
+export class AppRoutingModule {}

integration/ng-add/src/app/app.component.spec.ts

@@ -1,16 +1,12 @@
-import { TestBed } from '@angular/core/testing';
-import { RouterTestingModule } from '@angular/router/testing';
-import { AppComponent } from './app.component';
+import {TestBed} from '@angular/core/testing';
+import {RouterTestingModule} from '@angular/router/testing';
+import {AppComponent} from './app.component';
 
 describe('AppComponent', () => {
   beforeEach(async () => {
     await TestBed.configureTestingModule({
-      imports: [
-        RouterTestingModule
-      ],
-      declarations: [
-        AppComponent
-      ],
+      imports: [RouterTestingModule],
+      declarations: [AppComponent],
     }).compileComponents();
   });
 
@@ -30,6 +26,8 @@ describe('AppComponent', () => {
     const fixture = TestBed.createComponent(AppComponent);
     fixture.detectChanges();
     const compiled = fixture.nativeElement as HTMLElement;
-    expect(compiled.querySelector('.content span')?.textContent).toContain('ng-add app is running!');
+    expect(compiled.querySelector('.content span')?.textContent).toContain(
+      'ng-add app is running!',
+    );
   });
 });

integration/ng-add/src/app/app.component.ts

@@ -1,9 +1,9 @@
-import { Component } from '@angular/core';
+import {Component} from '@angular/core';
 
 @Component({
   selector: 'app-root',
   templateUrl: './app.component.html',
-  styleUrls: ['./app.component.scss']
+  styleUrls: ['./app.component.scss'],
 })
 export class AppComponent {
   title = 'ng-add';

integration/ng-add/src/app/app.module.ts

@@ -1,18 +1,13 @@
-import { NgModule } from '@angular/core';
-import { BrowserModule } from '@angular/platform-browser';
+import {NgModule} from '@angular/core';
+import {BrowserModule} from '@angular/platform-browser';
 
-import { AppRoutingModule } from './app-routing.module';
-import { AppComponent } from './app.component';
+import {AppRoutingModule} from './app-routing.module';
+import {AppComponent} from './app.component';
 
 @NgModule({
-  declarations: [
-    AppComponent
-  ],
-  imports: [
-    BrowserModule,
-    AppRoutingModule
-  ],
+  declarations: [AppComponent],
+  imports: [BrowserModule, AppRoutingModule],
   providers: [],
-  bootstrap: [AppComponent]
+  bootstrap: [AppComponent],
 })
-export class AppModule { }
+export class AppModule {}

integration/ng-add/src/environments/environment.prod.ts

@@ -1,3 +1,3 @@
 export const environment = {
-  production: true
+  production: true,
 };

integration/ng-add/src/environments/environment.ts

@@ -3,7 +3,7 @@
 // The list of file replacements can be found in `angular.json`.
 
 export const environment = {
-  production: false
+  production: false,
 };
 
 /*

integration/ng-add/src/main.ts

@@ -1,12 +1,13 @@
-import { enableProdMode } from '@angular/core';
-import { platformBrowserDynamic } from '@angular/platform-browser-dynamic';
+import {enableProdMode} from '@angular/core';
+import {platformBrowserDynamic} from '@angular/platform-browser-dynamic';
 
-import { AppModule } from './app/app.module';
-import { environment } from './environments/environment';
+import {AppModule} from './app/app.module';
+import {environment} from './environments/environment';
 
 if (environment.production) {
   enableProdMode();
 }
 
-platformBrowserDynamic().bootstrapModule(AppModule)
+platformBrowserDynamic()
+  .bootstrapModule(AppModule)
   .catch(err => console.error(err));

integration/ng-add/src/polyfills.ts

@@ -52,8 +52,7 @@
 /***************************************************************************************************
  * Zone JS is required by default for Angular itself.
  */
-import 'zone.js';  // Included with Angular CLI.
-
+import 'zone.js'; // Included with Angular CLI.
 
 /***************************************************************************************************
  * APPLICATION IMPORTS

integration/ng-add/src/test.ts

@@ -1,25 +1,27 @@
 // This file is required by karma.conf.js and loads recursively all the .spec and framework files
 
 import 'zone.js/testing';
-import { getTestBed } from '@angular/core/testing';
+import {getTestBed} from '@angular/core/testing';
 import {
   BrowserDynamicTestingModule,
-  platformBrowserDynamicTesting
+  platformBrowserDynamicTesting,
 } from '@angular/platform-browser-dynamic/testing';
 
 declare const require: {
-  context(path: string, deep?: boolean, filter?: RegExp): {
+  context(
+    path: string,
+    deep?: boolean,
+    filter?: RegExp,
+  ): {
     keys(): string[];
     <T>(id: string): T;
   };
 };
 
 // First, initialize the Angular testing environment.
-getTestBed().initTestEnvironment(
-  BrowserDynamicTestingModule,
-  platformBrowserDynamicTesting(),
-  { teardown: { destroyAfterEach: true }},
-);
+getTestBed().initTestEnvironment(BrowserDynamicTestingModule, platformBrowserDynamicTesting(), {
+  teardown: {destroyAfterEach: true},
+});
 
 // Then we find all the tests.
 const context = require.context('./', true, /\.spec\.ts$/);