chore(ci): bump actions/upload-artifact from 4.6.2 to 7.0.1#36
chore(ci): bump actions/upload-artifact from 4.6.2 to 7.0.1#36dependabot[bot] wants to merge 1 commit into
Conversation
LabelsThe following labels could not be found: Please fix the above issues or remove invalid values from |
Jerry-Xin
left a comment
There was a problem hiding this comment.
Summary: This PR is relevant to the project and safely updates the pinned actions/upload-artifact version in the Docker publish workflow.
β Highlights
- Project relevance gate: passes. The change affects
.github/workflows/docker-publish.yml, which is part of this repositoryβs release pipeline. .github/workflows/docker-publish.yml:207: The action remains pinned to an exact commit SHA, matching the repositoryβs existing supply-chain hardening pattern..github/workflows/docker-publish.yml:209-212: Existing inputs (name,path,if-no-files-found,retention-days) remain valid forupload-artifact@v7.0.1..github/workflows/docker-publish.yml:165-169: The upload runs on GitHub-hosted runners, so the new Node 24 runtime requirement for v7 should not block this workflow..github/workflows/docker-publish.yml:222-227: The downstreamdownload-artifactusage is still compatible with the uploaded digest artifacts.
No π΄ Critical issues found. No test changes are required for this dependency-only workflow pin update; validation will come from the next Docker publish workflow run.
lml2468
left a comment
There was a problem hiding this comment.
[APPROVE β COMMENT due to self-review restriction] Independent cross-review.
Verified:
- SHA pin integrity β
043fb46d1a93c77aae656e7c1c64a875d1fc6a0amatchesactions/upload-artifacttagv7.0.1exactly (lightweight tag, direct commit ref). β - Usage compatibility β The
withparameters used (name,path,if-no-files-found,retention-days) are stable across v4βv7. The Artifacts v2 wire format is unchanged. β - Paired action β
download-artifactat line 223 is still pinned to v4.3.0. PR #38 (bump download-artifact from 4.3.0 to 8.0.1) is already open to handle this. Both should merge for consistency, but v4 download is wire-compatible with v7 upload so merging this first is safe. β - CI β All checks green including
welcome / welcome(PR #34 fix took effect). β
No blocking issues. Standard Dependabot version bump, SHA verified.
lml2468
left a comment
There was a problem hiding this comment.
[APPROVE β self-review, posting as COMMENT] Correct SHA-pinned bump.
β
upload-artifact 4.6.2 β 7.0.1: SHA updated from ea165f8 β 043fb46, version comment updated.
download-artifact 4β8): upload-artifact and download-artifact must use a compatible artifact storage backend. v4 upload/download share the same backend; v7 upload introduces direct-upload support. Recommend merging #36 and #38 together in the same batch, or verifying cross-version compatibility in the release notes before merging individually.
No other issues. Ready to merge alongside #38.
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4.6.2 to 7.0.1. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@ea165f8...043fb46) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-version: 7.0.1 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
c4b7de9 to
506e6b8
Compare
Bumps actions/upload-artifact from 4.6.2 to 7.0.1.
Release notes
Sourced from actions/upload-artifact's releases.
... (truncated)
Commits
043fb46Merge pull request #797 from actions/yacaovsnc/update-dependency634250cInclude changes in typespec/ts-http-runtime 0.3.5e454baaReadme: bump all the example versions to v7 (#796)74fad66Update the readme with direct upload details (#795)bbbca2dSupport direct file uploads (#764)589182cUpgrade the module to ESM and bump dependencies (#762)47309c9Merge pull request #754 from actions/Link-/add-proxy-integration-tests02a8460Add proxy integration testb7c566aMerge pull request #745 from actions/upload-artifact-v6-releasee516bc8docs: correct description of Node.js 24 support in README