Skip to content

fix: Add Xcode selection step to CI jobs for macOS 26#5

Merged
isaaclins merged 2 commits into
mainfrom
dev
Dec 19, 2025
Merged

fix: Add Xcode selection step to CI jobs for macOS 26#5
isaaclins merged 2 commits into
mainfrom
dev

Conversation

@isaaclins

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings December 19, 2025 01:02

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 adds Xcode selection configuration to four CI jobs running on macOS 26 runners. The changes attempt to ensure a specific Xcode version (26.0) is used during CI execution by setting the DEVELOPER_DIR environment variable and running xcode-select commands.

  • Adds DEVELOPER_DIR environment variable pointing to Xcode 26.0
  • Adds explicit Xcode selection step using xcode-select before running tests
  • Applies changes consistently across quicktest-macos, unit-test-macos, build-macos, and perf-test-macos jobs

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/ci.yml Outdated
Comment on lines +65 to +68
- name: Select Xcode 26
run: |
sudo xcode-select -s /Applications/Xcode_26.0.app
xcodebuild -version

Copilot AI Dec 19, 2025

Copy link

Choose a reason for hiding this comment

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

The DEVELOPER_DIR environment variable points to /Applications/Xcode_26.0.app/Contents/Developer, but the xcode-select command uses /Applications/Xcode_26.0.app (without /Contents/Developer). These should be consistent - xcode-select expects the path to the .app bundle, while DEVELOPER_DIR should point to the Developer directory. However, since DEVELOPER_DIR is already set correctly, the xcode-select path should also include /Contents/Developer or the DEVELOPER_DIR setting makes the xcode-select call redundant.

Copilot uses AI. Check for mistakes.
Comment thread .github/workflows/ci.yml Outdated
Comment on lines +91 to +94
- name: Select Xcode 26
run: |
sudo xcode-select -s /Applications/Xcode_26.0.app
xcodebuild -version

Copilot AI Dec 19, 2025

Copy link

Choose a reason for hiding this comment

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

The DEVELOPER_DIR environment variable points to /Applications/Xcode_26.0.app/Contents/Developer, but the xcode-select command uses /Applications/Xcode_26.0.app (without /Contents/Developer). These should be consistent - xcode-select expects the path to the .app bundle, while DEVELOPER_DIR should point to the Developer directory. However, since DEVELOPER_DIR is already set correctly, the xcode-select path should also include /Contents/Developer or the DEVELOPER_DIR setting makes the xcode-select call redundant.

Copilot uses AI. Check for mistakes.
Comment thread .github/workflows/ci.yml Outdated
Comment on lines +117 to +120
- name: Select Xcode 26
run: |
sudo xcode-select -s /Applications/Xcode_26.0.app
xcodebuild -version

Copilot AI Dec 19, 2025

Copy link

Choose a reason for hiding this comment

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

The DEVELOPER_DIR environment variable points to /Applications/Xcode_26.0.app/Contents/Developer, but the xcode-select command uses /Applications/Xcode_26.0.app (without /Contents/Developer). These should be consistent - xcode-select expects the path to the .app bundle, while DEVELOPER_DIR should point to the Developer directory. However, since DEVELOPER_DIR is already set correctly, the xcode-select path should also include /Contents/Developer or the DEVELOPER_DIR setting makes the xcode-select call redundant.

Copilot uses AI. Check for mistakes.
Comment thread .github/workflows/ci.yml Outdated
Comment on lines +146 to +149
- name: Select Xcode 26
run: |
sudo xcode-select -s /Applications/Xcode_26.0.app
xcodebuild -version

Copilot AI Dec 19, 2025

Copy link

Choose a reason for hiding this comment

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

The DEVELOPER_DIR environment variable points to /Applications/Xcode_26.0.app/Contents/Developer, but the xcode-select command uses /Applications/Xcode_26.0.app (without /Contents/Developer). These should be consistent - xcode-select expects the path to the .app bundle, while DEVELOPER_DIR should point to the Developer directory. However, since DEVELOPER_DIR is already set correctly, the xcode-select path should also include /Contents/Developer or the DEVELOPER_DIR setting makes the xcode-select call redundant.

Copilot uses AI. Check for mistakes.
@isaaclins isaaclins merged commit bcf05c7 into main Dec 19, 2025
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