Merge pull request #69 from mbland/fix-download-artifact

Add actions/checkout to publish-test-results
mbland · Dec 19, 2023 · ec8346a · ec8346a
2 parents dc4f4fa + 4dfb9b9
commit ec8346a
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/.github/workflows/publish-test-results.yaml b/.github/workflows/publish-test-results.yaml
@@ -19,6 +19,16 @@ jobs:
   report:
     runs-on: ubuntu-latest
     steps:
+      # dorny/test-reporter has an `artifact:` attribute that will check out the
+      # repo and download test results automatically. However,
+      # dorny/test-reporter@v1 wasn't compatible with the bump to
+      # actions/upload-artifact@v4 in run-tests.yaml. So we manually check
+      # out the repo and download the results to avoid the rev lock.
+      - name: Check out repository
+        uses: actions/checkout@v4
+        with:
+          ref: ${{ github.event.workflow_run.head_commit.id }}
+
       - name: Download test results
         uses: actions/download-artifact@v4
         with:

diff --git a/.github/workflows/run-tests.yaml b/.github/workflows/run-tests.yaml
@@ -14,7 +14,8 @@ jobs:
     name: "Run Tests"
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
+      - name: Check out repository
+        uses: actions/checkout@v4
 
       - name: Set up Java
         uses: actions/setup-java@v4