Skip to content

Conversation

thomasvl
Copy link
Collaborator

@thomasvl thomasvl commented Dec 5, 2023

Enable the feature on all targets, and then go through doing the fixits within Xcode 15 that are triggered as a result.

This basically replaces #1509

fyi - @mrabiciu

@thomasvl thomasvl requested review from FranzBusch and tbkka December 5, 2023 17:11
Enable the feature on all targets, and then go through doing the fixits within
Xcode 15 that are triggered as a result.

This basically replaces apple#1509
@thomasvl thomasvl force-pushed the upcomingFeature_ExistentialAny branch from 132d1b5 to ab4ce58 Compare December 5, 2023 17:14
@thomasvl
Copy link
Collaborator Author

thomasvl commented Dec 5, 2023

My (foolish) original intent was just to add the flags and land that, didn't realize just how many things would have to be updated at the same time, hence this ended up being a replacement for the other PR (with the feature also getting enabled).

There are a few other targets we haven't enabled StrictConcurrency=complete, they end up needing some code changes, so I left that disable on the targets. Might be something worth following up on at some point for completeness.

@thomasvl thomasvl merged commit 0f3e371 into apple:main Dec 6, 2023
@thomasvl thomasvl deleted the upcomingFeature_ExistentialAny branch December 6, 2023 13:51
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