Repository navigation
Conversation
GitHub is retiring the macos-14 runner image. No other hosted image ships an iOS 17 simulator runtime, so drop the iOS 17.5 / Xcode 15.4 entry from both test matrices. The macos-15 and macos-26 entries remain. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Remove older build matrix configurations .github/workflows/ci.yml |
The build-and-test and build-example-app matrices no longer include the macOS 14 / iOS 17.5 / Xcode 15.4 configuration. |
Estimated code review effort: 1 (Trivial) | ~3 minutes
Change: Other
Suggested reviewers: patrickcroninmm
Merge Risk: 
75618
-
The retiring runner is removed while two build configurations remain. No actionable merge risk is established.
Pre-merge checks: 
-
Check name Status Explanation Description Check Check skipped - CodeRabbit’s high-level summary is enabled. Title check The title clearly and concisely describes the main change: removing use of the macos-14 runner from the CI matrices. Docstring Coverage No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat… Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request. Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files.
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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.
A rabbit checks the matrix rows,
One older runner now departs.
Two newer targets stay in place,
The workflow hops along its way.
I nibble greens and close the page.
Comment @coderabbitai help to get the list of available commands.
GitHub retires the
macos-14runner image on 2026-11-02, and brownouts are already running. This removes themacos-14/ iOS 17.5 / Xcode 15.4 entry from thebuild-and-testandbuild-example-appmatrices. Themacos-15(iOS 18.5) andmacos-26(iOS 26.5) entries stay.The entry cannot move to a newer image. The
macos-15andmacos-26images ship no iOS 17 simulator runtime and no Xcode 15.4. The oldest iOS simulator runtime onmacos-15is 18.5.Reviewer decision: CI now tests iOS 18.5 and later, while
Package.swiftstill declares iOS 15 as the minimum. Is that coverage gap acceptable, or should CI install an older simulator runtime withxcodebuild -downloadPlatform iOS? That download adds several minutes to each job.🤖 Generated with Claude Code
Summary by CodeRabbit