Skip to content

Drop pinned OS versions from CI simulator destinations#11

Merged
BrentMifsud merged 3 commits intomainfrom
bm/fix-ci-destinations
Mar 2, 2026
Merged

Drop pinned OS versions from CI simulator destinations#11
BrentMifsud merged 3 commits intomainfrom
bm/fix-ci-destinations

Conversation

@BrentMifsud
Copy link
Copy Markdown
Owner

Summary

  • Removes OS:26.1 and arch:arm64 from xcodebuild destination specifiers in the CI workflow
  • xcodebuild will automatically resolve to whatever simulator runtime is installed on the runner, preventing failures when GitHub updates Xcode images

This fixes the flaky iOS CI failure we saw on PR #10 where iPhone 17 with OS:26.1 wasn't available.

Test plan

  • CI should pass on all 6 platforms with the relaxed destinations

🤖 Generated with Claude Code

Removes OS:26.1 and arch:arm64 from xcodebuild destination specifiers
so the CI automatically picks whatever simulator runtime is available
on the runner, preventing failures when GitHub updates Xcode.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@BrentMifsud BrentMifsud force-pushed the bm/fix-ci-destinations branch 2 times, most recently from 5003496 to 21550c1 Compare March 2, 2026 02:00
Replace hardcoded simulator names and OS versions with dynamic
discovery using xcrun simctl. Each simulator platform job now
queries available devices at runtime and picks the first match,
making CI resilient to runner image changes.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@BrentMifsud BrentMifsud force-pushed the bm/fix-ci-destinations branch from 21550c1 to 5bd1a05 Compare March 2, 2026 02:04
Split the single "Run tests" step into two conditional steps
(simulator vs non-simulator) to avoid an empty else branch when
matrix.command is unset for simulator platforms.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@BrentMifsud BrentMifsud merged commit 08bec91 into main Mar 2, 2026
12 checks passed
@BrentMifsud BrentMifsud deleted the bm/fix-ci-destinations branch March 2, 2026 02:13
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.

1 participant