-
-
Notifications
You must be signed in to change notification settings - Fork 45
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
Use async
/await
and actors instead of Combine
#283
Conversation
We should remember to add the concurrency warnings to CartonKit. swiftSettings: [
.unsafeFlags([
"-Xfrontend", "-warn-concurrency",
"-Xfrontend", "-enable-actor-data-race-checks",
])
] Probably too early now as SwiftPM, TSC & Vapor types are not yet annotated with Sendable. |
Not merging this yet, seeing some problems with the watcher not picking up changes... |
I've been working with main branch of carton and lately seeing similar issues as well. Not sure if a regression from before but I guess it's not introduced by this PR. I think we can merge and look at the file watcher separately. |
This makes our codebase smaller by ~80 lines and arguably more readable. Also removes OpenCombine dependency.