Upgrade to print-diff@2 (microsoft#1469)

Author: saschanaz

.github/workflows/ci.yml

@@ -6,8 +6,8 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-node@v2
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
         with:
           node-version: "lts/*"
           cache: npm

.github/workflows/codeowners-merge.yml

@@ -9,7 +9,7 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v1
+      - uses: actions/checkout@v3
       - name: Run Codeowners merge check
         uses: OSS-Docs-Tools/[email protected]
         if: github.repository == 'microsoft/TypeScript-DOM-lib-generator'

.github/workflows/deploy.yml

@@ -11,8 +11,8 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-node@v2
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
         with:
           node-version: "lts/*"
           cache: npm
@@ -25,7 +25,7 @@ jobs:
       - name: Create packages for .d.ts files
         run: node deploy/createTypesPackages.js
 
-      - name: "Deploy built packages to NPM" 
+      - name: "Deploy built packages to NPM"
         run: |
           echo "//registry.npmjs.org/:_authToken=${NPM_TOKEN}" > ~/.npmrc
           npm i -g npm@latest

.github/workflows/pr-to-typescript.yml

@@ -7,8 +7,8 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-node@v2
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
         with:
           node-version: "lts/*"
           cache: npm

.github/workflows/test_typescript.yml

@@ -13,8 +13,8 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-node@v2
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
         with:
           node-version: "lts/*"
           cache: npm

.github/workflows/update-core-deps.yml

@@ -11,8 +11,8 @@ jobs:
   update-webref:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-node@v2
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
         with:
           node-version: "lts/*"
           cache: npm
@@ -22,6 +22,10 @@ jobs:
       - run: ncu -u
       - run: npm i
       - run: git restore package.json
+      # package-lock wants to remember the original version numbers of package.json
+      # This prevents annoying change when contributors run `npm i` on their local machine.
+      # Example: https://github.com/microsoft/TypeScript-DOM-lib-generator/pull/1463
+      - run: npm i
       - id: build
         run: npm run build && npm run baseline-accept
         continue-on-error: true

deploy/deployChangedPackages.js

@@ -8,7 +8,7 @@
 import * as fs from "fs";
 import { spawnSync } from "child_process";
 import { Octokit } from "@octokit/rest";
-import printDiff from "print-diff";
+import { printInlineDiff } from "print-diff";
 import { generateChangelogFrom } from "../lib/changelog.js";
 import { packages } from "./createTypesPackages.js";
 import { fileURLToPath } from "node:url";
@@ -65,7 +65,7 @@ for (const dirName of fs.readdirSync(generatedDir)) {
       );
       console.log(` - ${file}`);
       if (oldFile !== generatedDTSContent)
-        printDiff(oldFile, generatedDTSContent);
+        printInlineDiff(oldFile, generatedDTSContent);
 
       const title = `## \`${file}\``;
       const notes = generateChangelogFrom(oldFile, generatedDTSContent);

package-lock.json

@@ -36,26 +36,26 @@
   "license": "Apache-2.0",
   "type": "module",
   "devDependencies": {
-    "@mdn/browser-compat-data": "^5.2.22",
+    "@mdn/browser-compat-data": "^5.2.26",
     "@octokit/rest": "^19.0.5",
-    "@types/node": "^18.11.17",
-    "@types/prettier": "^2.7.1",
+    "@types/node": "^18.11.18",
+    "@types/prettier": "^2.7.2",
     "@types/webidl2": "^23.13.6",
-    "@typescript-eslint/eslint-plugin": "^5.46.1",
-    "@typescript-eslint/parser": "^5.46.1",
-    "@webref/css": "^6.1.1",
+    "@typescript-eslint/eslint-plugin": "^5.48.1",
+    "@typescript-eslint/parser": "^5.48.1",
+    "@webref/css": "^6.2.0",
     "@webref/elements": "^2.1.0",
-    "@webref/idl": "^3.22.4",
-    "bcd-idl-mapper": "^2.2.1",
+    "@webref/idl": "^3.23.0",
+    "bcd-idl-mapper": "^2.2.2",
     "cpx2": "^4.2.0",
-    "danger": "^11.2.0",
-    "eslint": "^8.30.0",
-    "eslint-config-prettier": "^8.5.0",
+    "danger": "^11.2.1",
+    "eslint": "^8.31.0",
+    "eslint-config-prettier": "^8.6.0",
     "eslint-plugin-prettier": "^4.2.1",
     "jsonc-parser": "^3.2.0",
     "node-fetch": "^3.3.0",
-    "prettier": "^2.8.1",
-    "print-diff": "^1.0.0",
+    "prettier": "^2.8.2",
+    "print-diff": "^2.0.0",
     "styleless-innertext": "^1.1.3",
     "typescript": "^4.9.4",
     "webidl2": "^24.2.2"

package.json

@@ -1,6 +1,6 @@
 import * as fs from "fs";
 import child_process from "child_process";
-import printDiff from "print-diff";
+import { printInlineDiff } from "print-diff";
 import { fileURLToPath } from "url";
 
 const baselineFolder = new URL("../baselines/", import.meta.url);
@@ -28,7 +28,7 @@ function compareToBaselines() {
     );
     if (baseline !== generated) {
       console.error(`Test failed: '${file}' is different from baseline file.`);
-      printDiff(baseline, generated);
+      printInlineDiff(baseline, generated);
       return false;
     }
   }