Skip to content

Conversation

@sosukesuzuki
Copy link
Member

@sosukesuzuki sosukesuzuki commented Nov 11, 2025

macos-13 machines will be deprecated so we should use macos-14 ( actions/runner-images#13046 ).

In fact, our build is failing due to this https://github.com/oven-sh/WebKit/actions/runs/19271032742/job/55099084552

@coderabbitai
Copy link

coderabbitai bot commented Nov 11, 2025

Walkthrough

Updated macOS runner specifications in GitHub Actions workflow from macos-13-* to macos-15-* across all job configurations (largest, xlarge, asan variants) in both Debug and Release builds.

Changes

Cohort / File(s) Summary
GitHub Actions workflow
\.github/workflows/build-reusable\.yml
Updated macOS runner specifications from macos-13-* to macos-15-* for all job variants (largest, xlarge, asan) in both Debug and Release configurations

Pre-merge checks

❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title states 'macos-14' but the actual changes update runners to 'macos-15', not 'macos-14'. The title is misleading about the specific version being updated to. Update the PR title to 'Use macos-15 instead of macos-13 in CI' to accurately reflect the changes made in the pull request.
Description check ⚠️ Warning The PR description lacks the required bug reference, Reviewed-by line, and detailed explanation format specified in the WebKit template. Add a Bugzilla bug reference (or note if this is infrastructure-only), include 'Reviewed by' line, and provide a structured explanation of the change following the WebKit PR template format.

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.


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

@sosukesuzuki sosukesuzuki changed the title Use macos-15 instead of macos-13 in CI Use macos-14 instead of macos-13 in CI Nov 12, 2025
@sosukesuzuki sosukesuzuki merged commit c3e10aa into main Nov 12, 2025
1 check 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