-
Notifications
You must be signed in to change notification settings - Fork 55
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
Add Sendable conformance to some basic SchemaTypes #322
Conversation
@bdbergeron: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/ |
👷 Deploy request for eclectic-pie-88a2ba pending review.Visit the deploys page to approve it
|
👷 Deploy request for apollo-ios-docc pending review.Visit the deploys page to approve it
|
This gets us partway to resolving apollographql/apollo-ios#3291. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution @bdbergeron.
4851f15b Add Sendable conformance to some basic SchemaTypes (#322) git-subtree-dir: apollo-ios git-subtree-split: 4851f15be36e778c4cfd72162c107f2777430354
…SchemaTypes git-subtree-dir: apollo-ios git-subtree-mainline: a60b3a4 git-subtree-split: 4851f15be36e778c4cfd72162c107f2777430354
Changes in Swift 5.10 and Xcode 15.3 improved the complete concurrency checking functionality in preparation for Swift 6.0. When using Apollo in a project with complete concurrency checking enabled, many warnings are surfaced for some of the generated schema types such as
Interface
,Object
, andUnion
. These types are simplestruct
s with no mutability semantics, so they can adoptSendable
without any further changes or concerns. The same goes forOperationDefinition
andOperationDocument
types used in generated queries, mutations, and fragments.