Skip to content

Conversation

@jumhyn-browser
Copy link
Contributor

Think these should all be fine?

Copy link
Member

@compnerd compnerd left a comment

Choose a reason for hiding this comment

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

CC: @parkera

@compnerd
Copy link
Member

@swift-ci please test

@compnerd
Copy link
Member

@swift-ci please test

@compnerd
Copy link
Member

@swift-ci please test Windows platform

@jumhyn-browser
Copy link
Contributor Author

Windows run succeeded but didn't get linked back here for some reason

@compnerd
Copy link
Member

@swift-ci please test Windows platform

@parkera parkera requested a review from jmschonfeld February 19, 2024 22:21
Copy link
Contributor

@jmschonfeld jmschonfeld left a comment

Choose a reason for hiding this comment

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

All looks fine to me, they're all trivial structs so they can definitely be Sendable

@compnerd compnerd merged commit eafffe1 into swiftlang:main Feb 21, 2024
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.

3 participants