Skip to content

Commit

Permalink
chore: bump the upload-artifact and download-artifact version (#566)
Browse files Browse the repository at this point in the history
[v3 of upload-artifact and download-artifact can no longer be
used](https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/).
This PR bumps the version to v4.

For upload, v4 differs from v3 in that v3 would allow uploading multiple
times with the same name and v4 does not. We do re-use the
snapcraft-logs name - but only in failure, so I *think* this means that
the workflow would exit before any clash occurred. We could adjust the
names to avoid that possibility - I'm guessing that we only manually
inspect them, rather than use the result in future work.
  • Loading branch information
tonyandrewmeyer committed Feb 3, 2025
1 parent 06294e9 commit d2fab76
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/binaries.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ jobs:
echo "PEBBLE_VERSION=${PEBBLE_VERSION}" >>$GITHUB_OUTPUT
- name: Upload archive as Actions artifact
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: ${{ steps.build.outputs.ARCHIVE_FILE }}
path: dist/${{ steps.build.outputs.ARCHIVE_FILE }}
Expand Down
12 changes: 6 additions & 6 deletions .github/workflows/snap.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ jobs:
id: build-snap
uses: snapcore/action-build@v1.3.0

- uses: actions/upload-artifact@v3
- uses: actions/upload-artifact@v4
if: failure()
with:
name: snapcraft-logs
Expand All @@ -52,7 +52,7 @@ jobs:
${{ env.SNAP_NAME }}_*.txt
- name: Attach ${{ steps.build-snap.outputs.snap }} to GH workflow execution
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: ${{ steps.build-snap.outputs.snap }}
path: ${{ steps.build-snap.outputs.snap }}
Expand Down Expand Up @@ -94,7 +94,7 @@ jobs:
with:
snapcraft-args: remote-build --build-for=${{ matrix.arch }} --launchpad-accept-public-upload

- uses: actions/upload-artifact@v3
- uses: actions/upload-artifact@v4
if: failure()
with:
name: snapcraft-logs
Expand All @@ -104,7 +104,7 @@ jobs:
${{ env.SNAP_NAME }}_*.txt
- name: Attach ${{ steps.build-snap.outputs.snap }} to GH workflow execution
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: steps.build-snap.outcome == 'success'
with:
name: ${{ steps.build-snap.outputs.snap }}
Expand All @@ -121,7 +121,7 @@ jobs:
(needs.local-build.result == 'success' || needs.local-build.result == 'skipped') &&
(needs.remote-build.result == 'success' || needs.remote-build.result == 'skipped')
steps:
- uses: actions/download-artifact@v3
- uses: actions/download-artifact@v4

- id: get-snap
run: echo "filename=$(find ${{ env.SNAP_NAME }}*${{ env.TEST_ON }}*snap/*)" >> $GITHUB_OUTPUT
Expand All @@ -146,7 +146,7 @@ jobs:
matrix:
arch: ["amd64", "arm64", "ppc64el", "armhf", "s390x"]
steps:
- uses: actions/download-artifact@v3
- uses: actions/download-artifact@v4

- id: get-snap
run: echo "filename=$(find ${{ env.SNAP_NAME }}*${{ matrix.arch }}*snap/*)" >> $GITHUB_OUTPUT
Expand Down

0 comments on commit d2fab76

Please sign in to comment.