build: update to latest version of shared dev-infra code

Updates to the latest version of the shared dev-infra code.
Also this commit adds `yargs` as a needed dependency for some
Bazel tooling exposed by the dev-infra package, notably the
http server rule.

with that, we also remove minimist in favor of Yargs, like we did
in other Angular repositories before as well.

(cherry picked from commit 52b3e9c)

Author: devversion

.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/commit-message-based-labels@613f706bc2467da4064f51be09ba0f8f1c3cb57e
+      - uses: angular/dev-infra/github-actions/commit-message-based-labels@8298e121c51960857ef39abc16b743775ff6be68
         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@613f706bc2467da4064f51be09ba0f8f1c3cb57e
+      - uses: angular/dev-infra/github-actions/lock-closed@8298e121c51960857ef39abc16b743775ff6be68
         with:
           lock-bot-key: ${{ secrets.LOCK_BOT_PRIVATE_KEY }}

package.json

@@ -74,7 +74,7 @@
     "@angular/bazel": "13.2.0-rc.1",
     "@angular/cli": "13.2.0-rc.1",
     "@angular/compiler-cli": "13.2.0-rc.1",
-    "@angular/dev-infra-private": "https://github.com/angular/dev-infra-private-builds.git#5847aa4e7ebe7de9fc772bf37e76bcdb8c8207a1",
+    "@angular/dev-infra-private": "https://github.com/angular/dev-infra-private-builds.git#2b0196f6ebf6de2e62836e962b9f42007e6cf5d6",
     "@angular/localize": "13.2.0-rc.1",
     "@angular/platform-browser-dynamic": "13.2.0-rc.1",
     "@angular/platform-server": "13.2.0-rc.1",
@@ -152,7 +152,6 @@
     "@types/jasmine": "^3.6.0",
     "@types/luxon": "^1.27.0",
     "@types/marked": "^2.0.0",
-    "@types/minimist": "^1.2.0",
     "@types/node": "^14.14.22",
     "@types/node-fetch": "^2.5.5",
     "@types/parse5": "^6.0.0",
@@ -162,6 +161,7 @@
     "@types/send": "^0.14.5",
     "@types/shelljs": "^0.8.9",
     "@types/yaml": "^1.9.7",
+    "@types/yargs": "^17.0.8",
     "autoprefixer": "^10.2.5",
     "browser-sync": "2.26.13",
     "chalk": "^4.1.0",
@@ -193,7 +193,6 @@
     "madge": "^4.0.0",
     "marked": "^2.0.0",
     "minimatch": "^3.0.4",
-    "minimist": "^1.2.0",
     "moment": "^2.18.1",
     "node-fetch": "^2.6.0",
     "parse5": "^6.0.1",
@@ -216,10 +215,11 @@
     "tsickle": "0.39.1",
     "tslint": "^6.1.3",
     "tsutils": "^3.21.0",
-    "typescript-4.4": "npm:[email protected]",
     "typescript": "~4.5.2",
+    "typescript-4.4": "npm:[email protected]",
     "vrsource-tslint-rules": "6.0.0",
-    "yaml": "^1.10.0"
+    "yaml": "^1.10.0",
+    "yargs": "^17.2.1"
   },
   "resolutions": {
     "@angular/dev-infra-private/typescript": "~4.5.2",

scripts/create-package-archives.js

@@ -12,17 +12,15 @@
 const {join} = require('path');
 const {rm, mkdir, test, ls, set, exec, cd} = require('shelljs');
 const {red, green} = require('chalk');
-const minimist = require('minimist');
+const yargs = require('yargs');
 
 const projectDir = join(__dirname, '../');
 const archivesDir = 'dist/release-archives';
 const releasesDir = 'dist/releases';
-let {suffix} = minimist(process.argv.slice(2), {string: ['suffix']});
-
-if (!suffix) {
-  console.error(red('No suffix specified. Pass one with --suffix <text>'));
-  process.exit(1);
-}
+const {suffix} = yargs(process.argv.slice(2))
+  .option('suffix', {type: 'string', demandOption: true})
+  .strict()
+  .parseSync();
 
 // Fail if any ShellJS command fails.
 set('-e');

scripts/run-component-tests.js

@@ -17,7 +17,7 @@
  *   --no-watch | Watch mode is enabled by default. This flag opts-out to standard Bazel.
  */
 
-const minimist = require('minimist');
+const yargs = require('yargs');
 const shelljs = require('shelljs');
 const chalk = require('chalk');
 const path = require('path');
@@ -35,15 +35,25 @@ shelljs.set('-e');
 shelljs.cd(projectDir);
 
 // Extracts the supported command line options.
-const {
-  _: components,
-  local,
-  firefox,
-  watch,
-} = minimist(args, {
-  boolean: ['local', 'firefox', 'watch'],
-  default: {watch: true},
-});
+const {components, local, firefox, watch} = yargs(args)
+  .command('* <components..>', 'Run tests for specified components', args =>
+    args.positional('components', {type: 'array'}),
+  )
+  .option('local', {
+    type: 'boolean',
+    description: 'Whether test should run in local mode. You can manually connect a browser then.',
+  })
+  .option('firefox', {
+    type: 'boolean',
+    description: 'Whether browser tests should run within Firefox.',
+  })
+  .option('watch', {
+    type: 'boolean',
+    default: true,
+    description: 'Whether tests should be re-run automatically upon changes.',
+  })
+  .strict()
+  .parseSync();
 
 // Whether tests for all components should be run.
 const all = components.length === 1 && components[0] === 'all';