-
Notifications
You must be signed in to change notification settings - Fork 73
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Swift 6 Compatibility #149
base: main
Are you sure you want to change the base?
Swift 6 Compatibility #149
Conversation
Does this PR still make sense? |
Yeah, it still makes sense. Just waiting on the Swift 6.0 PR in the upstream |
Looks like Swift 6.0 is finally available with swift-actions/setup-swift? |
171c5c3
to
767ca7f
Compare
This test is creates unreliable CI since we can't ensure performance of the runner machine.
@xtremekforever Good reminder - thanks! I've rebased this branch, and I think it's ready to go now! Unfortunately I couldn't get rid of all the Swift 6 warnings since we're still targeting a few Swift 5 versions that don't have the replacement APIs. But we should be able to remove those as we push our version support forward over time. |
This removes warnings and adds Swift 6 compatibility CI checks.