Skip to content

Fix github action for testing iOS - #3562

Merged
pasin merged 1 commit into
release/4.1from
fix-gha-4.1
Aug 19, 2026
Merged

Fix github action for testing iOS#3562
pasin merged 1 commit into
release/4.1from
fix-gha-4.1

Conversation

@pasin

@pasin pasin commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Remove Xcode 16.4 selection and simulator device lookup step

- Remove Xcode 16.4 selection and simulator device lookup step
@github-actions

github-actions Bot commented Aug 18, 2026

Copy link
Copy Markdown

This is a release branch and commits are restricted.

Please confirm this PR is one of the following:

  • A response to a customer ask
  • A change per our security policy
  • A non-functional change (e.g. changes needed for building an older version)
  • A bug fix for a x.y.0 release
  • A change that has been granted an exception (please comment)

@pasin
pasin requested review from jianminzhao and a lite review from Copilot August 18, 2026 23:25

Copilot AI 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.

Pull request overview

This PR updates the iOS GitHub Actions workflow to make CI builds less dependent on hardcoded Xcode and simulator device selection, relying instead on Xcode’s generic iOS Simulator destination for xcodebuild build-for-testing.

Changes:

  • Removed the step that forced selection of a specific Xcode install (/Applications/Xcode_16.4.app).
  • Removed the simulator device discovery/file handoff step (xctrace ... > device).
  • Updated the xcodebuild build-for-testing invocation to use -destination "generic/platform=iOS Simulator".

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

@pasin
pasin merged commit 05b37a6 into release/4.1 Aug 19, 2026
10 checks passed
@pasin
pasin deleted the fix-gha-4.1 branch August 19, 2026 02:31
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.

3 participants