Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: Update actions/download-artifact and actions/upload-artifact to v4 #2615

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yaroslav-007
Copy link
Contributor

Description:
This PR is addressing the following note:

Warning

actions/upload-artifact@v3 is scheduled for deprecation on November 30, 2024. [Learn more.](https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/) Similarly, v1/v2 are scheduled for deprecation on June 30, 2024. Please update your workflow to use v4 of the artifact actions. This deprecation will not impact any existing versions of GitHub Enterprise Server being used by customers.

It has addressed the Breaking Changes in v4 where is needed:

Uploading to the same named Artifact multiple times.

Due to how Artifacts are created in this new version, it is no longer possible to upload to the same named Artifact multiple times. You must either split the uploads into multiple Artifacts with different names, or only upload once. Otherwise you will encounter an error.

Related issue(s):

Fixes #2596

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Signed-off-by: Yaroslav Markovski <yaroslav.markovski@limechain.tech>
@yaroslav-007 yaroslav-007 self-assigned this Jun 21, 2024
Copy link

sonarcloud bot commented Jun 21, 2024

@yaroslav-007 yaroslav-007 requested a review from isavov June 21, 2024 10:08
Copy link

Tests

    2 files  152 suites   14s ⏱️
847 tests 846 ✔️ 1 💤 0
859 runs  858 ✔️ 1 💤 0

Results for commit 57a64d0.

Copy link

Acceptance Tests

  18 files  255 suites   31m 25s ⏱️
603 tests 598 ✔️ 3 💤 2
832 runs  822 ✔️ 7 💤 3

Results for commit 57a64d0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update upload-artifact/download-artifact versions for test reports
1 participant