Skip to content

25236: Upgrades deprecated GitHub Actions - #32

Merged
apbassett merged 1 commit into
mainfrom
25236-upgrade-deprecated-actions
Jul 9, 2026
Merged

25236: Upgrades deprecated GitHub Actions#32
apbassett merged 1 commit into
mainfrom
25236-upgrade-deprecated-actions

Conversation

@apbassett

Copy link
Copy Markdown
Member

No description provided.

@apbassett
apbassett requested review from a team as code owners July 9, 2026 18:23
@apbassett
apbassett enabled auto-merge (squash) July 9, 2026 18:23

@howso-mario howso-mario left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Reviewed by Hermes Agent (howso-mario) at head aecb2e2.

Verdict: APPROVE (advisory).

I reviewed the workflow-only diff upgrading docker/setup-qemu-action v3->v4, docker/setup-buildx-action v3->v4, docker/metadata-action v5->v6, docker/login-action v3->v4, and docker/build-push-action v5->v7. The new major tags exist upstream, and the existing inputs used by this workflow remain compatible for both non-release PR builds and workflow_dispatch release builds. No security regression found in the changed lines.

Validation:

  • GitHub PR checks: build (3.11), build (3.12), build (3.13), build (3.14) passed. Release-only jobs were skipped as expected for this PR path.
  • Local: PyYAML parsed .github/workflows/build.yml and enumerated expected jobs.
  • Local: git diff --check origin/main...HEAD passed.
  • Local: verified the new Docker action tags exist with git ls-remote.
  • Dedicated review lane: Codex review found no clear breakage introduced by the patch.

No follow-up required from me.

@apbassett
apbassett merged commit ce01df1 into main Jul 9, 2026
10 checks passed
@apbassett
apbassett deleted the 25236-upgrade-deprecated-actions branch July 9, 2026 18:29
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