chore: make danger work with packaged api

microsoft · Jan 10, 2023 · 9adb7e0 · 9adb7e0
1 parent de8f843
commit 9adb7e0
Show file tree

Hide file tree

Showing 5 changed files with 25 additions and 5 deletions.
diff --git a/azure-pipelines.yml b/azure-pipelines.yml
@@ -53,7 +53,7 @@ jobs:
 
       ## Danger.js checks for Fluent UI N*
       - script: |
-          yarn danger ci
+          DANGER_DISABLE_TRANSPILATION="true" yarn danger ci
         displayName: danger
         condition: eq(variables.isPR, true)
         env:

diff --git a/dangerfile.ts → dangerfile.js b/dangerfile.ts → dangerfile.js
@@ -1,14 +1,15 @@
-import { danger, fail, warn, markdown, message } from 'danger';
+// @ts-check
 
-import { detectNonApprovedDependencies, checkChangelog, checkStorybookVersions } from '@fluentui/scripts-dangerjs';
+const { danger, fail, warn, markdown, message } = require('danger');
+const { detectNonApprovedDependencies, checkChangelog, checkStorybookVersions } = require('@fluentui/scripts-dangerjs');
 
 /**
  * This trick (of explicitly passing Danger JS utils as function arg, instead of importing them at places where needed)
  * is necessary due to the magic DangerJS is doing with imports: https://spectrum.chat/danger/javascript/danger-js-actually-runs-your-imports-as-globals~0a005b56-31ec-4919-9a28-ced623949d4d
  */
 const dangerJS = { danger, fail, warn, markdown, message };
 
-export default async () => {
+module.exports = async () => {
   await checkChangelog(dangerJS);
   await detectNonApprovedDependencies(dangerJS);
   await checkStorybookVersions(dangerJS);

diff --git a/scripts/dangerjs/package.json b/scripts/dangerjs/package.json
@@ -2,7 +2,7 @@
   "name": "@fluentui/scripts-dangerjs",
   "version": "0.0.1",
   "private": true,
-  "main": "src/index.ts",
+  "main": "src/index.js",
   "scripts": {
     "format": "prettier -w --ignore-path ../../.prettierignore .",
     "format:check": "yarn format -c",

diff --git a/scripts/dangerjs/src/index.ts → scripts/dangerjs/src/api.ts b/scripts/dangerjs/src/index.ts → scripts/dangerjs/src/api.ts
diff --git a/scripts/dangerjs/src/index.js b/scripts/dangerjs/src/index.js
@@ -0,0 +1,19 @@
+// require('@fluentui/scripts-babel/register');
+
+require('@babel/register')({
+  extensions: ['.js', '.ts', '.tsx'],
+  ignore: [/node_modules/],
+  presets: [
+    [
+      '@babel/preset-env',
+      {
+        loose: true,
+        // modules: 'cjs',
+        targets: { node: 'current' },
+      },
+    ],
+    ['@babel/preset-typescript'],
+  ],
+});
+
+module.exports = require('./api');