automerge / automerge-repo-swift

Extends the Automerge-swift library, providing support for working with multiple Automerge documents at once, with pluggable network and storage providers.
https://swiftpackageindex.com/automerge/automerge-repo-swift/documentation/automergerepo
MIT License
23 stars 4 forks source link

Use TaskGroup to wait on network activating or retry timeout, whichever is first #102

Closed jessegrosjean closed 5 months ago

jessegrosjean commented 5 months ago

I think this addresses #87

jessegrosjean commented 5 months ago

@heckj Also, if you have any idea how to get rid of the for try await _ in group (and see commented code above) let me know. I haven't done much work with async and can't see why one works without warning and the other doesn't.

heckj commented 5 months ago

failing test appears to be a flake. I'm going to briefly hold merging this to lock down where that flake is happening, then merge this (assuming it is unrelated, which i'll check as i dig it out when I get home)