Repository navigation
build(deps): bump actions/upload-artifact from 7.0.1 to 7.0.2 - #50
Merged
qawatake merged 1 commit intoOct 17, 2026
Merged
Conversation
Owner
|
@dependabot rebase |
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 7.0.1 to 7.0.2. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@043fb46...cf430e0) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-version: 7.0.2 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
Bot
force-pushed
the
dependabot/github_actions/actions/upload-artifact-7.0.2
branch
from
October 17, 2026 03:13
834f710 to
6b06276
Compare
qawatake
approved these changes
Oct 17, 2026
qawatake
left a comment
Owner
There was a problem hiding this comment.
Update review: actions/upload-artifact v7.0.1 → v7.0.2
Final decision: Proceed
Risk assessment: Medium — axes hit: test coverage (the changed step only runs if: failure(), so this repo's CI does not execute it; coverage relies on upstream CI of the exact commit). All other axes Low.
Review type: Approve
Release/deploy: Not required (no-release) — rationale: the PR only changes .github/workflows/ci.yml. The shipped artifact (main.js + manifest.json) is built locally and attached to GitHub releases; no workflow builds or publishes it, and this action is not on the bundled / emitted / generated path.
Changes reviewed (2.1)
- Primary sources consulted: release notes v7.0.2; compare v7.0.1...v7.0.2 (4 commits: PR #821 bump
@actions/artifactto 6.3.1, PR #822 prepare release); upstream toolkit PRs actions/toolkit#2534 (honorRetry-Afteron HTTP 429, 6.3.0) and #2537 (declare protobuf RPC runtime dependency, 6.3.1). - Version range reviewed: v7.0.1 → v7.0.2 (no intermediate release).
- Breaking / behavior changes: none documented. Behavior change: retries on HTTP 429 now honor valid
Retry-After. Changed files:dist/upload/index.js,dist/merge/index.js,package.json,package-lock.json,.licenses/.action.ymlis not changed, so inputs/outputs are identical.
Impact and adequacy of the PR (2.2, 2.3)
| Change | Affected location / no impact | Evidence | Is the PR adequate |
|---|---|---|---|
429 retry / Retry-After handling |
ci.yml jobs e2e and test-e2e-tools, failure-only diagnostic upload of test-results/ |
grep upload-artifact .github/workflows/ci.yml: 2 usages, both if: failure() with name, path, retention-days: 7, if-no-files-found: ignore |
Yes (both pins updated, comment # v7.0.2 matches) |
@actions/artifact 6.3.1 (dependency declaration fix) |
Bundled into the action's dist; no repo-side impact |
action.yml unchanged; inputs used here unchanged |
Yes |
Verification (2.4)
- CI results, and what CI does not cover: this PR was rebased (
@dependabot rebase) ontomain599803eafter #51 (jdx/mise-action v5.1.1) was merged and its post-merge CI passed. On the new head6b06276(parent599803e, diff still only the two upload-artifact pins, mise-action pins stay at v5.1.1), run 38997989233 —build,e2e,test-e2e-toolsall success (e2e1 passed). Earlier run on the pre-rebase head834f710: 38944522153, also all success. The log shows the action atcf430e0...was downloaded, but the step was skipped because nothing failed. The changed code path is not exercised by this repo's CI. - Additional checks I did: upstream checks on commit
cf430e0(Build on ubuntu/macos/windows, which upload and download artifacts,check-dist,Check licenses, CodeQL,test-proxy-upload) all success. Inputs used here are unchanged inaction.yml.
Rollback, supply chain, license (2.5, 2.6, 2.7)
- Rollback: revert the SHA pin. No irreversible side effects.
- Supply chain: tag
v7.0.2→ commitcf430e030ddbb5b0abf93d22962f4752f3646cd9(lightweight, matches the pin), identical toactions/upload-artifactmainHEAD. Release published 2026-10-07 bytunc-d, whereas v7.0.1 was published byyacaovsnc. Why that is acceptable: both PRs (#821, #822) came from branches insideactions/upload-artifact, so the author has write access there. Both were approved byGhadimiR, a long-time actions maintainer. The author also has merged PRs inactions/toolkitfor the same@actions/artifactchange.@actions/artifact@6.3.1itself is published via npm trusted publishing with provenance. Neither v7.0.1 nor v7.0.2 is an immutable release, but this repo pins by commit SHA, so a re-pointed tag cannot reach this workflow. No issues opened in actions/upload-artifact since 2026-10-06. Published 10 days ago. - License: MIT → MIT (unchanged).
Blocking conditions
- Hit: none. The verification for this change is upstream's CI on the exact commit plus an unchanged
action.ymlinterface. I classify the lack of in-repo execution as a known constraint. If this step breaks, the only effect is a missing diagnostic artifact on a run that has already failed.
Requests
- None.
Unverified items / notes for other reviewers (2.9)
- Evidence reused across the rebase: release notes, compare, tag → SHA, upstream checks and license were collected for the same pinned commit
cf430e0, which did not change with the rebase. Diff, CI and labels were re-checked on the new head. - Unverified: this step running in this repo with v7.0.2. It will run the next time an e2e job fails. If the upload then errors, revert this pin.
- Unverified: whether
tunc-dis a member of theactionsorg (membership is not public). The trust argument above rests on repository write access and maintainer approval, not on membership.
dependabot
Bot
deleted the
dependabot/github_actions/actions/upload-artifact-7.0.2
branch
October 17, 2026 03:16
This was referenced Oct 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps actions/upload-artifact from 7.0.1 to 7.0.2.
Release notes
Sourced from actions/upload-artifact's releases.
Commits
cf430e0Merge pull request #822 from actions/tunc-d-prepare-upload-v7-0-2c2acdedchore: prepare v7.0.2 release4687c1bMerge pull request #821 from actions/tunc-d-upload-artifact-package-bump8cbe1beBump@actions/artifactto 6.3.1