Skip to content

[Concurrency][Distributed] Move AnyActor to Concurrency module #41722

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

Merged
merged 2 commits into from
Mar 8, 2022

Conversation

ktoso
Copy link
Contributor

@ktoso ktoso commented Mar 8, 2022

Move AnyActor to concurrency module, now that SE-0336 was accepted

@ktoso ktoso requested a review from DougGregor March 8, 2022 06:30
@ktoso
Copy link
Contributor Author

ktoso commented Mar 8, 2022

@swift-ci please test

@ktoso ktoso added concurrency Feature: umbrella label for concurrency language features distributed Feature → concurrency: distributed actor labels Mar 8, 2022
These are not an issue because adding a @_marker protocol conformance has no ABI impact - the protocol is not available at runtime.
@ktoso
Copy link
Contributor Author

ktoso commented Mar 8, 2022

Had to ignore the ABI test -- because AnyActor is a @_marker protocol it has no ABI impact.

Please confirm though @DougGregor @kavon ?

@ktoso
Copy link
Contributor Author

ktoso commented Mar 8, 2022

@swift-ci please test

@ktoso ktoso requested a review from kavon March 8, 2022 11:08
@ktoso ktoso merged commit 033b8bf into swiftlang:main Mar 8, 2022
@ktoso ktoso deleted the wip-move-anyactor branch March 8, 2022 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
concurrency Feature: umbrella label for concurrency language features distributed Feature → concurrency: distributed actor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants