ci/cd: centralize and bump artifact uploads

- Upgrade `actions/upload-artifact` to `v4` to address deprecation warnings related to Node.js 16, improving compatibility with GitHub runners. This resolves the following warning from the runners: > Node.js 16 actions are deprecated. Please update the following actions > to use Node.js 20: actions/upload-artifact@v3. - Centralize the use of the `upload-artifact` action through a new custom action, improving maintainability and consistency across workflows.
undergroundwires · May 28, 2024 · 22d6c79 · 22d6c79
1 parent 795b7f0
commit 22d6c79
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 3 deletions.
diff --git a/.github/actions/upload-artifact/action.yaml b/.github/actions/upload-artifact/action.yaml
@@ -0,0 +1,15 @@
+inputs:
+  name:
+    required: true
+  path:
+    required: true
+
+runs:
+  using: composite
+  steps:
+    -
+      name: Upload artifact
+      uses: actions/upload-artifact@v4
+      with:
+        name: ${{ inputs.name }}
+        path: ${{ inputs.path }}
diff --git a/.github/workflows/checks.desktop-runtime-errors.yaml b/.github/workflows/checks.desktop-runtime-errors.yaml
@@ -70,7 +70,7 @@ jobs:
       -
         name: Upload screenshot
         if: always() # Run even if previous step fails
-        uses: actions/upload-artifact@v3
+        uses: ./.github/actions/upload-artifact
         with:
           name: screenshot-${{ matrix.os }}
           path: screenshot.png
diff --git a/.github/workflows/tests.e2e.yaml b/.github/workflows/tests.e2e.yaml
@@ -51,14 +51,14 @@ jobs:
       -
         name: Upload screenshots
         if: failure() # Run only if previous steps fail because screenshots will be generated only if E2E test failed
-        uses: actions/upload-artifact@v3
+        uses: ./.github/actions/upload-artifact
         with:
           name: e2e-screenshots-${{ matrix.os }}
           path: ${{ steps.artifacts.outputs.SCREENSHOTS_DIR }}
       -
         name: Upload videos
         if: always() # Run even if previous steps fail because test run video is always captured
-        uses: actions/upload-artifact@v3
+        uses: ./.github/actions/upload-artifact
         with:
           name: e2e-videos-${{ matrix.os }}
           path: ${{ steps.artifacts.outputs.VIDEOS_DIR }}