Merge pull request #809 from edoardopirovano/include-run-id

Include matrix information in debug artifact name
github · Nov 4, 2021 · ea16943 · ea16943
2 parents 0442e71 + 375c147
commit ea16943
Show file tree

Hide file tree

Showing 5 changed files with 17 additions and 5 deletions.
diff --git a/.github/workflows/__debug-artifacts.yml b/.github/workflows/__debug-artifacts.yml
diff --git a/lib/analyze-action.js b/lib/analyze-action.js
diff --git a/lib/analyze-action.js.map b/lib/analyze-action.js.map
diff --git a/pr-checks/checks/debug-artifacts.yml b/pr-checks/checks/debug-artifacts.yml
@@ -13,7 +13,7 @@ steps:
     id: analysis
   - uses: actions/download-artifact@v2
     with:
-      name: debug-artifacts
+      name: debug-artifacts-${{ matrix.os }}-${{ matrix.version }}
   - shell: bash
     run: |
       LANGUAGES="cpp csharp go java javascript python"

diff --git a/src/analyze-action.ts b/src/analyze-action.ts
@@ -237,8 +237,14 @@ async function run() {
 }
 
 async function uploadDebugArtifacts(toUpload: string[], rootDir: string) {
+  let suffix = "";
+  const matrix = actionsUtil.getRequiredInput("matrix");
+  if (matrix !== undefined && matrix !== "null") {
+    for (const entry of Object.entries(JSON.parse(matrix)).sort())
+      suffix += `-${entry[1]}`;
+  }
   await artifact.create().uploadArtifact(
-    DEBUG_ARTIFACT_NAME,
+    `${DEBUG_ARTIFACT_NAME}${suffix}`,
     toUpload.map((file) => path.normalize(file)),
     path.normalize(rootDir)
   );