Skip to content

chore(deps): update github artifact actions (major) - #29

Open
renovate[bot] wants to merge 1 commit into
mainfrom
renovate/major-github-artifact-actions
Open

chore(deps): update github artifact actions (major)#29
renovate[bot] wants to merge 1 commit into
mainfrom
renovate/major-github-artifact-actions

Conversation

@renovate

@renovate renovate Bot commented Oct 24, 2025

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Type Update Change
actions/download-artifact action major v5v8
actions/upload-artifact action major v4v7

Release Notes

actions/download-artifact (actions/download-artifact)

v8.0.1

Compare Source

What's Changed

Full Changelog: actions/download-artifact@v8...v8.0.1

v8.0.0

Compare Source

v8 - What's new

[!IMPORTANT]
actions/download-artifact@​v8 has been migrated to an ESM module. This should be transparent to the caller but forks might need to make significant changes.

[!IMPORTANT]
Hash mismatches will now error by default. Users can override this behavior with a setting change (see below).

Direct downloads

To support direct uploads in actions/upload-artifact, the action will no longer attempt to unzip all downloaded files. Instead, the action checks the Content-Type header ahead of unzipping and skips non-zipped files. Callers wishing to download a zipped file as-is can also set the new skip-decompress parameter to true.

Enforced checks (breaking)

A previous release introduced digest checks on the download. If a download hash didn't match the expected hash from the server, the action would log a warning. Callers can now configure the behavior on mismatch with the digest-mismatch parameter. To be secure by default, we are now defaulting the behavior to error which will fail the workflow run.

ESM

To support new versions of the @​actions/* packages, we've upgraded the package to ESM.

What's Changed

Full Changelog: actions/download-artifact@v7...v8.0.0

v8

Compare Source

v7.0.0

Compare Source

v7 - What's new

[!IMPORTANT]
actions/download-artifact@​v7 now runs on Node.js 24 (runs.using: node24) and requires a minimum Actions Runner version of 2.327.1. If you are using self-hosted runners, ensure they are updated before upgrading.

Node.js 24

This release updates the runtime to Node.js 24. v6 had preliminary support for Node 24, however this action was by default still running on Node.js 20. Now this action by default will run on Node.js 24.

What's Changed

New Contributors

Full Changelog: actions/download-artifact@v6.0.0...v7.0.0

v7

Compare Source

v6.0.0

Compare Source

What's Changed

BREAKING CHANGE: this update supports Node v24.x. This is not a breaking change per-se but we're treating it as such.

New Contributors

Full Changelog: actions/download-artifact@v5...v6.0.0

v6

Compare Source

actions/upload-artifact (actions/upload-artifact)

v7.0.1

Compare Source

What's Changed

Full Changelog: actions/upload-artifact@v7...v7.0.1

v7.0.0

Compare Source

v7 What's new

Direct Uploads

Adds support for uploading single files directly (unzipped). Callers can set the new archive parameter to false to skip zipping the file during upload. Right now, we only support single files. The action will fail if the glob passed resolves to multiple files. The name parameter is also ignored with this setting. Instead, the name of the artifact will be the name of the uploaded file.

ESM

To support new versions of the @actions/* packages, we've upgraded the package to ESM.

What's Changed

New Contributors

Full Changelog: actions/upload-artifact@v6...v7.0.0

v7

Compare Source

v6.0.0

Compare Source

v6 - What's new

[!IMPORTANT]
actions/upload-artifact@​v6 now runs on Node.js 24 (runs.using: node24) and requires a minimum Actions Runner version of 2.327.1. If you are using self-hosted runners, ensure they are updated before upgrading.

Node.js 24

This release updates the runtime to Node.js 24. v5 had preliminary support for Node.js 24, however this action was by default still running on Node.js 20. Now this action by default will run on Node.js 24.

What's Changed

Full Changelog: actions/upload-artifact@v5.0.0...v6.0.0

v6

Compare Source

v5.0.0

Compare Source

What's Changed

BREAKING CHANGE: this update supports Node v24.x. This is not a breaking change per-se but we're treating it as such.

New Contributors

Full Changelog: actions/upload-artifact@v4...v5.0.0

v5

Compare Source


Configuration

📅 Schedule: (UTC)

  • Branch creation
    • At any time (no schedule defined)
  • Automerge
    • At any time (no schedule defined)

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@claude

claude Bot commented Dec 12, 2025

Copy link
Copy Markdown

Code Review - PR #29: Update GitHub Artifact Actions (Major)

Summary

This PR updates GitHub Actions dependencies:

  • actions/upload-artifact: v4 → v6 (skipping v5)
  • actions/download-artifact: v5 → v7 (skipping v6)

✅ Positive Aspects

  1. Automated Dependency Management: Good use of Renovate bot for keeping dependencies current
  2. Consistent Updates: All usages of the actions are updated uniformly across the workflow
  3. Security & Performance: Major version updates typically include important security patches and performance improvements

⚠️ Concerns & Recommendations

1. Breaking Changes - Major Version Jumps

Severity: High

This PR includes major version upgrades which may introduce breaking changes:

  • Upload artifact: v4 → v6 (skipping v5 entirely)
  • Download artifact: v5 → v7 (skipping v6 entirely)

Recommendation:

  • Review the release notes for v5, v6 of upload-artifact and v6, v7 of download-artifact
  • Common breaking changes in these actions include:
    • Changes to artifact retention policies
    • Modified artifact naming conventions
    • Updated Node.js runtime requirements
    • Changes to artifact upload/download behavior with duplicate names
  • Test the publish workflow in a safe environment before merging (e.g., tag a pre-release to verify the workflow works)

2. Lack of Testing for Publish Workflow

Severity: Medium

The publish.yml workflow is only triggered on tag pushes (refs/tags/). There's no way to validate these changes work correctly without creating an actual tag.

Recommendation:

  • Consider adding a test job that validates artifact upload/download without publishing
  • Or manually test by creating a test tag on a fork/branch before merging

3. Missing CI Validation

Severity: Medium

The changes don't appear to be validated by the existing CI workflow (.github/workflows/ci.yml), which doesn't use artifact upload/download actions.

Recommendation:

  • The publish workflow should be tested independently or
  • Add a comment documenting that this was validated (if already tested)

4. Incomplete Package Name Configuration

Severity: Low (Pre-existing issue, not introduced by this PR)

Lines 41 and 105 of publish.yml contain placeholder text:

url: https://pypi.org/p/<package-name>
url: https://test.pypi.org/p/<package-name>

Recommendation:

  • These should be updated to altair-upset for proper environment URLs
  • This is a separate issue but should be addressed

🔒 Security Review

  • ✅ No new security concerns introduced
  • ✅ Permissions remain appropriately scoped (id-token: write, contents: write)
  • ✅ Trusted publishing pattern continues to be used correctly

🧪 Test Coverage

  • ⚠️ No automated tests for the publish workflow
  • ⚠️ Manual verification recommended before merge

📋 Action Items Before Merge

  1. Critical: Review release notes for breaking changes:

  2. Recommended: Test the workflow:

    • Option A: Create a test tag on a fork to verify the workflow succeeds
    • Option B: Add a workflow_dispatch trigger temporarily to test manually
  3. Optional: Fix the placeholder <package-name> URLs (can be separate PR)

Overall Assessment

Status: ⚠️ Approve with Caution

The changes are straightforward and well-contained, but due to the major version jumps and lack of automated testing for the publish workflow, I recommend:

  1. Manual verification of the workflow before merging
  2. Reviewing GitHub Actions changelog for breaking changes
  3. Having a rollback plan ready if the first publish after merge fails

The Renovate bot did its job correctly, but human review of breaking changes is essential for major version updates.


🤖 Generated with Claude Code

@renovate
renovate Bot force-pushed the renovate/major-github-artifact-actions branch from 99a3c23 to 66d0673 Compare February 26, 2026 20:43
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.

0 participants