Skip to content

Repair wp/7.0 branch Actions: pin runner images instead of *-latest - #82805

Open
desrosj wants to merge 1 commit into
fix-actions-7.0/remove-cpu-cores-dependencyfrom
fix-actions-7.0/pin-runner-images
Open

desrosj wants to merge 1 commit into
fix-actions-7.0/remove-cpu-cores-dependencyfrom
fix-actions-7.0/pin-runner-images

Conversation

@desrosj

@desrosj desrosj commented Sep 11, 2026

Copy link
Copy Markdown
Member

Part of #82782.

What?

Replaces the floating ubuntu-latest, windows-latest, and macos-latest GitHub Actions runner labels in this branch's workflows with pinned versions: ubuntu-24.04, windows-2025, and macos-15 respectively.

Why?

GitHub has been emitting deprecation warnings for the floating -latest runner image aliases. Upstream trunk made this same change in PR #71201 (commit 82768d0e018b44007e3e3a25636c13375b33e7e7), "Avoid using *-latest for runner images", merged 2025-08-28. This PR ports the same substitution to wp/7.0.

Use of AI Tools

This PR was authored with the help of Claude Code.

@coderabbitai

coderabbitai Bot commented Sep 11, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: d726566f-5f80-4afb-971f-f402392a5ce8

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The runner substitution is valid, complete, and consistent with the upstream change.

Pull request overview

Pins the Slack notification workflow to a stable Ubuntu runner image.

Changes:

  • Replaces ubuntu-latest with ubuntu-24.04.
File summaries
File Description
.github/workflows/notify-slack-on-label.yml Pins the notification job’s runner image.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@desrosj
desrosj force-pushed the fix-actions-7.0/pin-runner-images branch from 5882e4a to 65ead4c Compare September 14, 2026 01:50
@desrosj
desrosj force-pushed the fix-actions-7.0/pin-runner-images branch from 65ead4c to 6033370 Compare September 14, 2026 02:34
@desrosj
desrosj force-pushed the fix-actions-7.0/pin-runner-images branch from 6033370 to f142139 Compare September 14, 2026 16:43
@desrosj
desrosj marked this pull request as ready for review September 14, 2026 16:57
@github-actions

Copy link
Copy Markdown

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: desrosj <desrosj@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@desrosj
desrosj force-pushed the fix-actions-7.0/pin-runner-images branch from f142139 to 8165606 Compare September 14, 2026 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Build Tooling Issues or PRs related to build tooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants