Skip to content

Conversation

@dependabot
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Sep 8, 2025

Bumps actions/download-artifact from 4 to 5.

Release notes

Sourced from actions/download-artifact's releases.

v5.0.0

What's Changed

v5.0.0

🚨 Breaking Change

This release fixes an inconsistency in path behavior for single artifact downloads by ID. If you're downloading single artifacts by ID, the output path may change.

What Changed

Previously, single artifact downloads behaved differently depending on how you specified the artifact:

  • By name: name: my-artifact → extracted to path/ (direct)
  • By ID: artifact-ids: 12345 → extracted to path/my-artifact/ (nested)

Now both methods are consistent:

  • By name: name: my-artifact → extracted to path/ (unchanged)
  • By ID: artifact-ids: 12345 → extracted to path/ (fixed - now direct)

Migration Guide

✅ No Action Needed If:
  • You download artifacts by name
  • You download multiple artifacts by ID
  • You already use merge-multiple: true as a workaround
⚠️ Action Required If:

You download single artifacts by ID and your workflows expect the nested directory structure.

Before v5 (nested structure):

- uses: actions/download-artifact@v4
  with:
    artifact-ids: 12345
    path: dist
# Files were in: dist/my-artifact/

Where my-artifact is the name of the artifact you previously uploaded

To maintain old behavior (if needed):

</tr></table> 

... (truncated)

Commits
  • 634f93c Merge pull request #416 from actions/single-artifact-id-download-path
  • b19ff43 refactor: resolve download path correctly in artifact download tests (mainly ...
  • e262cbe bundle dist
  • bff23f9 update docs
  • fff8c14 fix download path logic when downloading a single artifact by id
  • 448e3f8 Merge pull request #407 from actions/nebuk89-patch-1
  • 47225c4 Update README.md
  • See full diff in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Summary by CodeRabbit

  • Chores
    • Updated CI workflow to use the latest artifact download action, ensuring compatibility with current GitHub infrastructure.
    • Improves reliability and stability of artifact retrieval during automated builds and releases.
    • Reduces risk of pipeline failures related to outdated dependencies.
    • Streamlines future maintenance by standardizing on current action versions.
    • No changes to features, performance, or user experience; this is maintenance-only.

Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 4 to 5.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@v4...v5)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-version: '5'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code labels Sep 8, 2025
@dependabot dependabot bot requested a review from a team as a code owner September 8, 2025 05:35
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Sep 8, 2025
@dependabot dependabot bot requested review from hanabi1224 and sudo-shashank and removed request for a team September 8, 2025 05:35
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 8, 2025

Walkthrough

Bumps the GitHub Actions step in .github/workflows/forest.yml to use actions/download-artifact v5 instead of v4 for the forest-${{ runner.os }} artifact within the calibnet-api-test-stateful job. No other workflow logic or steps were modified.

Changes

Cohort / File(s) Summary
CI workflow action version bump
.github/workflows/forest.yml
Updated uses: actions/download-artifact@v4 to @v5 for the forest-${{ runner.os }} artifact download step in calibnet-api-test-stateful.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • hanabi1224
  • sudo-shashank

Warning

Review ran into problems

🔥 Problems

Errors were encountered while retrieving linked issues.

Errors (1)
  • Failed to retrieve linked issues from the platform client.
✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dependabot/github_actions/actions/download-artifact-5

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

🧹 Nitpick comments (5)
.github/workflows/forest.yml (5)

394-398: Remove duplicate download step to cut runner time and I/O.

This second actions/download-artifact@v5 call re-downloads the same artifact to the same path immediately after the previous step. It’s redundant and costs minutes.

Apply:

-      - uses: actions/download-artifact@v5
-        with:
-          name: "forest-${{ runner.os }}"
-          path: ~/.cargo/bin

147-155: Duplicate artifact download in calibnet-check.

Two consecutive downloads of the same artifact to the same path. Remove one to save time.

-      - uses: actions/download-artifact@v5
-        with:
-          name: "forest-${{ runner.os }}"
-          path: ~/.cargo/bin

177-185: Duplicate artifact download in calibnet-stateless mode check.

Same redundancy; safe to delete one.

-      - uses: actions/download-artifact@v5
-        with:
-          name: "forest-${{ runner.os }}"
-          path: ~/.cargo/bin

204-211: Duplicate artifact download in calibnet stateless RPC check.

Remove the extra download.

-      - uses: actions/download-artifact@v5
-        with:
-          name: "forest-${{ runner.os }}"
-          path: ~/.cargo/bin

365-372: Duplicate artifact download in calibnet eth mapping check.

Delete the redundant step.

-      - uses: actions/download-artifact@v5
-        with:
-          name: "forest-${{ runner.os }}"
-          path: ~/.cargo/bin
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between aa977cf and c6b7627.

📒 Files selected for processing (1)
  • .github/workflows/forest.yml (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (7)
  • GitHub Check: tests-release
  • GitHub Check: tests
  • GitHub Check: All lint checks
  • GitHub Check: Build forest binaries on Linux AMD64
  • GitHub Check: Build Ubuntu
  • GitHub Check: cargo-publish-dry-run
  • GitHub Check: Build MacOS
🔇 Additional comments (2)
.github/workflows/forest.yml (2)

390-398: Bump to actions/download-artifact@v5 is safe here (download by name).

You download a single artifact by name into a specific path, which retains the same extract-to-path behavior in v5. The v5 breaking change only affects single downloads by ID (now extracted directly to path instead of a nested subdir). No action needed. (github.com)


390-398: All download-artifact@v5 calls use the name/path syntax; no artifact-ids, pattern, or merge-multiple keys detected in any workflow.

@LesnyRumcajs LesnyRumcajs added this pull request to the merge queue Sep 8, 2025
Merged via the queue into main with commit ed3dd9b Sep 8, 2025
41 of 60 checks passed
@LesnyRumcajs LesnyRumcajs deleted the dependabot/github_actions/actions/download-artifact-5 branch September 8, 2025 07:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants