Quick / Nimble

A Matcher Framework for Swift and Objective-C
https://quick.github.io/Nimble/documentation/nimble/
Apache License 2.0
4.81k stars 598 forks source link

Replace public usage of DispatchTimeInterval with a new NimbleTimeInterval #1042

Closed younata closed 1 year ago

younata commented 1 year ago

NimbleTimeInterval is effectively the same as DispatchTimeInterval, except that it conforms to Sendable, and there is no never case. NimbleTimeInterval values are converted to DispatchTimeInterval values only when they actually need to be used with other Dispatch APIs.

This is a simple enough change that we can and should ship it prior to enabling concurrency checking.

This breaks the public API (a type signature changed, you can no longer use the never case with toEventually et al. Not that that was a good idea anyway.).

Resolves https://github.com/Quick/Nimble/issues/1031