Skip to content

Update CI to use Xcode 16.4 and 26.0.1#306

Merged
skorulis-ap merged 2 commits intomainfrom
luis/update-ci-for-swift-6.1-6.2
Oct 20, 2025
Merged

Update CI to use Xcode 16.4 and 26.0.1#306
skorulis-ap merged 2 commits intomainfrom
luis/update-ci-for-swift-6.1-6.2

Conversation

@luispadron
Copy link
Copy Markdown
Collaborator

@luispadron luispadron commented Oct 16, 2025

This updates CI to use a manual xcode-select and base tests on the Xcode version. This removes the need for a custom setup-swift action which can be stale/unmaintained (see: swift-actions/setup-swift#767)

Currently tests on Xcode 16.4 and Xcode 26.0.1

@luispadron luispadron force-pushed the luis/update-ci-for-swift-6.1-6.2 branch from 63d1744 to 4d1e926 Compare October 16, 2025 22:58
@luispadron
Copy link
Copy Markdown
Collaborator Author

@skorulis-ap @bradfol could yall review + update the required checks in github to match the new ones?

Screenshot 2025-10-16 at 6 59 58 PM

@luispadron luispadron enabled auto-merge October 16, 2025 23:00
@luispadron luispadron changed the title Use Swift 6.1. and 6.2 for CI, drop 6.0 Update CI to use Xcode 16.4 and 26.0.1 Oct 16, 2025
@luispadron luispadron force-pushed the luis/update-ci-for-swift-6.1-6.2 branch from 4d1e926 to 142db4b Compare October 16, 2025 23:03
@skorulis-ap
Copy link
Copy Markdown
Contributor

Looks like there's something causing the tests to fail but it's hard to see what the actual issue is. These CI jobs have been causing problems for a while and changes that work fine locally cause the compiler to crash, but I haven't had time to dig into what the actual issue is.

@luispadron
Copy link
Copy Markdown
Collaborator Author

@skorulis-ap yeah seems like some signal is killing the tests in 26.0.1. do we want to just merge this and #305 to unblock consumers from using new swift-syntax and then fix CI

@skorulis-ap
Copy link
Copy Markdown
Contributor

@skorulis-ap yeah seems like some signal is killing the tests in 26.0.1. do we want to just merge this and #305 to unblock consumers from using new swift-syntax and then fix CI

#305 is merged, that should be all you need to unblock right?

@skorulis-ap skorulis-ap force-pushed the luis/update-ci-for-swift-6.1-6.2 branch from 142db4b to cf936e8 Compare October 20, 2025 03:58
@skorulis-ap skorulis-ap disabled auto-merge October 20, 2025 05:23
@skorulis-ap skorulis-ap merged commit 38a23ff into main Oct 20, 2025
2 checks passed
@skorulis-ap skorulis-ap deleted the luis/update-ci-for-swift-6.1-6.2 branch October 20, 2025 05:23
@skorulis-ap
Copy link
Copy Markdown
Contributor

I figured out the failing test and disabled it in this PR. I've updated the branch rules to now wait for these jobs instead

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