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.

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';