Skip to content

Conversation

@phlogistonjohn
Copy link
Collaborator

manually update upload-artifact to v5 with pinned hashes

I noticed that dependabot was only updating a v4 -> v5, which is weird since I thought we either converted or removed all uses of unpinned versions.

Either way, use the latest with the pinned hashes now.

@phlogistonjohn phlogistonjohn added the no-API This PR does not include any changes to the public API of a go-ceph package label Nov 2, 2025
Copy link
Collaborator

@anoopcs9 anoopcs9 left a comment

Choose a reason for hiding this comment

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

lgtm.

@anoopcs9
Copy link
Collaborator

anoopcs9 commented Nov 3, 2025

@Mergifyio rebase

Manually update upload-artifact to v5.0.0
(330a01c490aca151604b8cf639adc76d48f6c5d4).

Signed-off-by: John Mulligan <jmulligan@redhat.com>
Manually update upload-artifact to v5.0.0
(330a01c490aca151604b8cf639adc76d48f6c5d4).
For some reason we had missed updating this to use a pinned hash.
Do so now.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
@mergify
Copy link

mergify bot commented Nov 3, 2025

rebase

✅ Branch has been successfully rebased

@anoopcs9 anoopcs9 force-pushed the jjm-upload-artifact-v5 branch from 8b16267 to 6d2f114 Compare November 3, 2025 08:56
@anoopcs9
Copy link
Collaborator

anoopcs9 commented Nov 3, 2025

@Mergifyio queue

@mergify
Copy link

mergify bot commented Nov 3, 2025

queue

🛑 GitHub permissions prevent queueing

Details

The PR cannot be merged because of GitHub restrictions. You must set up a update_bot_account on your queue_rules to automatically queue and merge PRs created by a GitHub App, or modify the GitHub Actions workflows.

@phlogistonjohn phlogistonjohn merged commit de97695 into ceph:master Nov 3, 2025
14 of 15 checks passed
@phlogistonjohn phlogistonjohn deleted the jjm-upload-artifact-v5 branch January 2, 2026 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-API This PR does not include any changes to the public API of a go-ceph package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants