build: update shared dev-infra package

.circleci/config.yml

@@ -387,7 +387,7 @@ jobs:
       - *setup_bazel_binary
 
       - run: yarn build-and-check-release-output
-      - run: yarn check-tools
+      - run: yarn check-tooling-setup
 
       # TODO(devversion): replace this with bazel tests that run Madge. This is
       # cumbersome and doesn't guarantee no circular deps for other entry-points.

.github/workflows/dev-infra.yml

@@ -9,6 +9,6 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - uses: angular/dev-infra/github-actions/breaking-changes-label@861dc90572784e714aeaa9dfb20ceebeb57cdb07
+      - uses: angular/dev-infra/github-actions/breaking-changes-label@3f4646c252bc54b2ae93b4e322ebde9030de30b2
         with:
           angular-robot-key: ${{ secrets.ANGULAR_ROBOT_PRIVATE_KEY }}

.github/workflows/lock-closed.yml

@@ -9,6 +9,6 @@ jobs:
   lock_closed:
     runs-on: ubuntu-latest
     steps:
-      - uses: angular/dev-infra/github-actions/lock-closed@7679cff885633cd33bf5ac7922a5304e8971a5a6
+      - uses: angular/dev-infra/github-actions/lock-closed@3f4646c252bc54b2ae93b4e322ebde9030de30b2
         with:
           lock-bot-key: ${{ secrets.LOCK_BOT_PRIVATE_KEY }}

.ng-dev/merge.ts

@@ -1,28 +1,22 @@
-import {DevInfraMergeConfig} from '@angular/dev-infra-private/ng-dev/pr/merge/config';
-import {getDefaultTargetLabelConfiguration} from '@angular/dev-infra-private/ng-dev/pr/merge/defaults';
-import {github} from './github';
-import {release} from './release';
+import {MergeConfig} from '@angular/dev-infra-private/ng-dev/pr/merge/config';
 
 /**
  * Configuration for the merge tool in `ng-dev`. This sets up the labels which
  * are respected by the merge script (e.g. the target labels).
  */
-export const merge: DevInfraMergeConfig['merge'] = async api => {
-  return {
-    // By default, the merge script merges locally with `git cherry-pick` and autosquash.
-    // This has the downside of pull requests showing up as `Closed` instead of `Merged`.
-    // In the components repository, since we don't use fixup or squash commits, we can
-    // 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'}
-      ]
-    },
-    claSignedLabel: 'cla: yes',
-    mergeReadyLabel: 'merge ready',
-    commitMessageFixupLabel: 'commit message fixup',
-    caretakerNoteLabel: 'caretaker note',
-    labels: await getDefaultTargetLabelConfiguration(api, github, release),
-  };
+export const merge: MergeConfig = {
+  // By default, the merge script merges locally with `git cherry-pick` and autosquash.
+  // This has the downside of pull requests showing up as `Closed` instead of `Merged`.
+  // In the components repository, since we don't use fixup or squash commits, we can
+  // 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'}
+    ]
+  },
+  claSignedLabel: 'cla: yes',
+  mergeReadyLabel: 'merge ready',
+  commitMessageFixupLabel: 'commit message fixup',
+  caretakerNoteLabel: 'caretaker note',
 };

.ng-dev/tsconfig.json

@@ -0,0 +1,10 @@
+{
+  "compilerOptions": {
+    "strict": true,
+    "target": "es2015",
+    "module": "commonjs",
+    "noEmit": true,
+    "skipLibCheck": true,
+    "types": []
+  },
+}

package.json

@@ -47,7 +47,7 @@
     "integration-tests:size-test": "bazel test //integration/size-test/...",
     "check-mdc-tests": "ts-node --project scripts/tsconfig.json scripts/check-mdc-tests.ts",
     "check-mdc-exports": "ts-node --project scripts/tsconfig.json scripts/check-mdc-exports.ts",
-    "check-tools": "yarn tsc --project tools/tsconfig-ci.json",
+    "check-tooling-setup": "yarn tsc --project tools/tsconfig-ci.json && yarn tsc --project .ng-dev/tsconfig.json",
     "tsc": "node ./node_modules/typescript/bin/tsc",
     "prepare": "husky install"
   },
@@ -75,7 +75,7 @@
     "@angular-devkit/schematics": "13.0.0-next.1",
     "@angular/bazel": "13.0.0-next.2",
     "@angular/compiler-cli": "13.0.0-next.2",
-    "@angular/dev-infra-private": "https://github.com/angular/dev-infra-private-builds.git#6254699cc7ccff62be948dc5edcdea87a936ff24",
+    "@angular/dev-infra-private": "https://github.com/angular/dev-infra-private-builds.git#56655ed6ccf24b7f3ad6b9ccdd1ef2896c7fa9f4",
     "@angular/localize": "13.0.0-next.2",
     "@angular/platform-browser-dynamic": "13.0.0-next.2",
     "@angular/platform-server": "13.0.0-next.2",

scripts/build-packages-dist.ts

@@ -72,7 +72,7 @@ function buildReleasePackages(useIvy: boolean, distPath: string,
   const targets = exec(queryPackagesCmd, true).split(/\r?\n/);
   const packageNames = getPackageNamesOfTargets(targets);
   const bazelBinPath = exec(`${bazelCmd} info bazel-bin`, true);
-  const getOutputPath = pkgName => join(bazelBinPath, 'src', pkgName, 'npm_package');
+  const getOutputPath = (pkgName: string) => join(bazelBinPath, 'src', pkgName, 'npm_package');
 
   // Build with "--config=release" or `--config=snapshot-build` so that Bazel
   // runs the workspace stamping script. The stamping script ensures that the

scripts/caretaking/resync-caretaker-app-prs.ts

@@ -1,4 +1,4 @@
-import * as Octokit from '@octokit/rest';
+import {Octokit} from '@octokit/rest';
 import * as fetch from 'node-fetch';
 
 const apiBaseUrl = 'https://test-jperrott.firebaseio.com/pulls';

scripts/check-mdc-exports-config.ts

@@ -21,5 +21,5 @@ export const config = {
       // Irrelevant for the MDC implementation, because the slider doesn't dispatch any events.
       'MatSliderChange'
     ]
-  }
+  } as {[entryPointName: string]: string[]}
 };

scripts/tsconfig.json

@@ -5,7 +5,12 @@
     "lib": ["es2016"],
     "moduleResolution": "node",
     "types": ["node"],
-    "strictNullChecks": true,
+    "strict": true,
+    "noEmit": true,
+    "skipLibCheck": true,
     "downlevelIteration": true
-  }
+  },
+  // The `firebase-functions` folder has its own `package.json` and does
+  // not pass type-checking. This excludes it from being checked.
+  "exclude": ["caretaking/firebase-functions/**"]
 }