Reenable danger in CI (DefinitelyTyped#74450)

Author: jakebailey

.github/workflows/CI.yml

@@ -124,38 +124,38 @@ jobs:
           retention-days: 1
         if: ${{ github.event_name == 'pull_request' }}
 
-  # dangerbot:
-  #   runs-on: ubuntu-latest
-  #   if: github.repository == 'DefinitelyTyped/DefinitelyTyped' && github.event_name == 'pull_request'
-
-  #   needs:
-  #     - test
-
-  #   steps:
-  #     - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
-  #     - uses: ./.github/actions/setup-for-scripts
-
-  #     - name: Get suggestions dir
-  #       id: suggestions-dir
-  #       run: echo "path=$(node ./scripts/get-suggestions-dir.js)" >> "$GITHUB_OUTPUT"
-
-  #     - uses: actions/download-artifact@37930b1c2abaa49bbe596cd826c3c89aef350131 # v7.0.0
-  #       with:
-  #         path: ${{ steps.suggestions-dir.outputs.path }}
-  #         merge-multiple: true
-
-  #     - name: 'Run Danger'
-  #       env:
-  #         # See https://github.com/danger/danger-js/issues/1042
-  #         DANGER_GITHUB_API_BASE_URL: 'https://api.github.com'
-
-  #       # Danger failing (for example through rate-limiting) shouldn't fail the build
-  #       run: |
-  #         # Exposing this token is safe because the user of it has no other public repositories
-  #         # and has no permission to modify this repository. See #62638 for the discussion.
-  #         TOKEN='ghp_i5wtj1l2AbpFv3OU96w6R'
-  #         TOKEN+='On3bHOkcV2AmVY6'
-  #         DANGER_GITHUB_API_TOKEN=$TOKEN pnpm danger ci || $( exit 0 )
+  dangerbot:
+    runs-on: ubuntu-latest
+    if: github.repository == 'DefinitelyTyped/DefinitelyTyped' && github.event_name == 'pull_request'
+
+    needs:
+      - test
+
+    steps:
+      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+      - uses: ./.github/actions/setup-for-scripts
+
+      - name: Get suggestions dir
+        id: suggestions-dir
+        run: echo "path=$(node ./scripts/get-suggestions-dir.js)" >> "$GITHUB_OUTPUT"
+
+      - uses: actions/download-artifact@37930b1c2abaa49bbe596cd826c3c89aef350131 # v7.0.0
+        with:
+          path: ${{ steps.suggestions-dir.outputs.path }}
+          merge-multiple: true
+
+      - name: 'Run Danger'
+        env:
+          # See https://github.com/danger/danger-js/issues/1042
+          DANGER_GITHUB_API_BASE_URL: 'https://api.github.com'
+
+        # Danger failing (for example through rate-limiting) shouldn't fail the build
+        run: |
+          # Exposing this token is safe because the user of it has no other public repositories
+          # and has no permission to modify this repository. See #62638 for the discussion.
+          TOKEN='ghp_i5wtj1l2AbpFv3OU96w6R'
+          TOKEN+='On3bHOkcV2AmVY6'
+          DANGER_GITHUB_API_TOKEN=$TOKEN pnpm danger ci || $( exit 0 )
 
   scripts:
     runs-on: ubuntu-latest

dangerfile.ts

@@ -1,9 +1,9 @@
-import fs = require("fs");
-import os = require("os");
-import path = require("path");
-import cp = require("child_process");
 import { mangleScopedPackage, suggestionsDir } from "@definitelytyped/utils";
 import { danger, fail, markdown } from "danger";
+import cp from "node:child_process";
+import fs from "node:fs";
+import os from "node:os";
+import path from "node:path";
 const lines: string[] = [];
 const missingProperty = /module exports a property named '(.+?)', which is missing/;
 
@@ -129,7 +129,7 @@ if (dprintErrors.length > 0) {
         "## Formatting errors",
         "",
         codeBlock,
-        ...dprintErrors.join("\n\n"),
+        dprintErrors.join("\n\n"),
         codeBlock,
     ];
 

package.json

@@ -37,7 +37,7 @@
         "@definitelytyped/header-parser": "latest",
         "@definitelytyped/typescript-versions": "latest",
         "@definitelytyped/utils": "latest",
-        "danger": "^11.2.3",
+        "danger": "^13.0.5",
         "dprint": "^0.49.0",
         "eslint-plugin-jsdoc": "^44.2.7",
         "husky": "^8.0.3",