Skip to content

Bump actions/upload-artifact to 7.0.1 - #301

Merged
floitsch merged 1 commit into
google:masterfrom
Marcono1234:bump-actions-upload-artifact
Aug 26, 2026
Merged

Bump actions/upload-artifact to 7.0.1#301
floitsch merged 1 commit into
google:masterfrom
Marcono1234:bump-actions-upload-artifact

Conversation

@Marcono1234

Copy link
Copy Markdown

See https://github.com/actions/upload-artifact/releases/tag/v7.0.1

For cifuzz.yml the previous pinned version (actions/upload-artifact@65d8626) was apparently not a release but just some commit around v3 (?), which is also not supported anymore. So if fuzzing had actually found something, it is quite likely that uploading the result would have failed.

And for scorecard.yml the workflow already logged a warning about Node.js 20 being deprecated.

Not sure why Dependabot had not attempted to update either of them so far. Maybe updates for that dependency had been ignored in the past (could try running @dependabot show actions/upload-artifact ignore conditions), or maybe the issue is the open PR limit:

open-pull-requests-limit: 3

@Marcono1234

Copy link
Copy Markdown
Author

The Zizmor error is unrelated to my changes; it is about the existing unpinned uses: github/codeql-action/upload-sarif (which also seems to cause issues for the other Dependabot PRs touching that file).

If you want I can fix that here as well though.

floitsch added a commit that referenced this pull request Aug 26, 2026
Pin the two mutable action references in the Scorecard workflow to the
exact commits currently selected by their version tags. Also correct the
stale get-cmake version comment so Zizmor can validate its existing hash
pin.

This provides a clean base for the currently failing action-update PRs
(#295, #296, #297, and #301). Their existing failures were produced
before these fixes.
For `cifuzz.yml` the previous pinned version was apparently not a release but
just some commit around v3 (?), which is also not supported anymore. So if
fuzzing had actually found something, it is quite likely that uploading the
result would have failed.

And for `scorecard.yml` the workflow already logged a warning about Node.js 20
being deprecated.

Not sure why Dependabot had not attempted to update either of them so far.
@floitsch
floitsch force-pushed the bump-actions-upload-artifact branch from 32ee8ce to ed7efb7 Compare August 26, 2026 13:03

@floitsch floitsch left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM.
Thanks.

@floitsch
floitsch merged commit 4aedac4 into google:master Aug 26, 2026
9 checks passed
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.

2 participants