Skip to content

Conversation

peterfriese
Copy link
Collaborator

Here are the changes required in the movie tracker sample app to make it work when compiling with Swift 6 language mode enabled.

Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
…eterfriese/samples/movietracker-new-ui/main

Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
…onnect' into peterfriese/samples/movietracker-new-ui/main

Signed-off-by: Peter Friese <[email protected]>
…terfriese/samples/movietracker

Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Signed-off-by: Peter Friese <[email protected]>
Copy link
Collaborator

@aashishpatil-g aashishpatil-g left a comment

Choose a reason for hiding this comment

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

The Swift 6 changes look fine.

In 11.4.0-beta (released yesterday, Nov 12 ) there are multiple changes to support Concurrency and we also released support in the codegen module last week.

Curious, if that works without making some of these changes - specifically adding @preconcurrency

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