Skip to content

Commit

Permalink
Merge pull request #532 from rspier/act4
Browse files Browse the repository at this point in the history
actions: Remove upload-artifact@v3 hack.
  • Loading branch information
rspier authored Jun 5, 2024
2 parents 7a66e85 + ff54f09 commit c75fe70
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 11 deletions.
3 changes: 3 additions & 0 deletions .github/workflows/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,3 +15,6 @@ $ act --artifact-server-path=/tmp/artifacts
You can use the `--reuse` flag to reuse information between runs,
which can speed things up. `--action-offline-mode` and `--pull=false`
can be used to test local copies of container images instead.

IMPORTANT: You need a version of act from 2024-05-20 or later.
(e.g. newer than v0.2.62) to support `actions/upload-artifact@v4`.
12 changes: 1 addition & 11 deletions .github/workflows/perl-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -73,18 +73,8 @@ jobs:
cpanm -n Devel::Cover::Report::Coveralls
cover -test -report Coveralls | tee -a $GITHUB_STEP_SUMMARY
continue-on-error: true
# We need different versions of upload-artifact in production or
# local testing due to https://github.com/nektos/act/issues/2135
- name: Archive plack logs (prod)
- name: Upload Artifacts
uses: actions/upload-artifact@v4
if: ${{ env.ACT != 'true' }}
with:
name: plack.log
path: |
/tmp/plack.log
- name: Archive plack logs (dev)
uses: actions/upload-artifact@v3
if: ${{ env.ACT == 'true' }}
with:
name: plack.log
path: |
Expand Down

0 comments on commit c75fe70

Please sign in to comment.