chore: only run install case in need (#67)

web-infra-dev · Nov 24, 2023 · 04a1233 · 04a1233
1 parent 3c9546d
commit 04a1233
Showing 1 changed file with 12 additions and 4 deletions.
diff --git a/scripts/src/pr-bench.ts b/scripts/src/pr-bench.ts
@@ -2,6 +2,7 @@ import { dev } from './runners/dev';
 import { build } from './runners/build';
 import {
   DefaultBenchCase,
+  ValidMetricsForCase,
   cloneRepo,
   getDataPath,
   mergeMetrics,
@@ -29,10 +30,17 @@ async function prBench() {
 
     await build(productName, caseName);
 
-    try {
-      await yarnInstall(productName, caseName);
-    } catch (err) {
-      console.log('failed to collect install size metrics:', err);
+    if (
+      ValidMetricsForCase[
+        caseName as keyof typeof ValidMetricsForCase
+      ]?.includes('installSize') ||
+      !ValidMetricsForCase[caseName as keyof typeof ValidMetricsForCase]
+    ) {
+      try {
+        await yarnInstall(productName, caseName);
+      } catch (err) {
+        console.log('failed to collect install size metrics:', err);
+      }
     }
 
     await mergeMetrics(productName, caseName);