Skip to content

Commit

Permalink
Upgrade upload-artifact action to v4, based on Node 20 (#14371)
Browse files Browse the repository at this point in the history
Even though V4 has breaking changes, particuarly around how
uploads with non-unique names are no longer automatically combined
this won't be a problem here, I think

Resolves #14147
  • Loading branch information
stevejalim committed Apr 2, 2024
1 parent 2e61f28 commit 87a3a41
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/cdn_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ jobs:
run: ./bin/integration_tests/cleanup_after_functional_tests.sh

- name: Store artifacts
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: test-results
path: results-${{github.run_id}}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/download_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ jobs:
run: ./bin/integration_tests/cleanup_after_functional_tests.sh

- name: Store artifacts
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: test-results
path: results-${{github.run_id}}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/integration_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ jobs:

- name: Store artifacts
if: always()
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: test-results
path: results-${{github.run_id}}-${{matrix.label}}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/pull_request_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ jobs:
CONTAINER_ID=$(docker ps -alq)
docker cp $CONTAINER_ID:/app/python_coverage .
- name: Store coverage as an artifact
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: coverage-results
path: python_coverage
Expand Down

0 comments on commit 87a3a41

Please sign in to comment.