Merge pull request #3209 from mashehu/fix-snapshot-artifact-upload

CI: fix artifact upload of snapshot report
nf-core · Oct 9, 2024 · dd21c7b · dd21c7b
2 parents 85a27ad + 058391d
commit dd21c7b
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/.github/workflows/pytest.yml b/.github/workflows/pytest.yml
@@ -142,18 +142,18 @@ jobs:
             exit 1
           fi
 
+      - name: remove slashes from test name
+        run: |
+          test=$(echo ${{ matrix.test }} | sed 's/\//__/g')
+          echo "test=${test}" >> $GITHUB_ENV
+
       - name: Store snapshot report
         uses: actions/upload-artifact@65462800fd760344b1a7b4382951275a0abb4808 # v4
         if: always()
         with:
-          name: Snapshot Report ${{ matrix.test }}
+          name: Snapshot Report ${{ env.test }}
           path: ./snapshot_report.html
 
-      - name: remove slashes from test name
-        run: |
-          test=$(echo ${{ matrix.test }} | sed 's/\//__/g')
-          echo "test=${test}" >> $GITHUB_ENV
-
       - name: Upload coverage
         uses: actions/upload-artifact@65462800fd760344b1a7b4382951275a0abb4808 # v4
         with: