swift-metrics icon indicating copy to clipboard operation
swift-metrics copied to clipboard

Published 20 hours ago • verified publisher icon apple

Should recordInterval live in CoreMetrics?

Open ktoso opened this issue 3 years ago • 1 comments

I initially said in review that it should be in Metrics: https://github.com/apple/swift-metrics/pull/83

but maybe that was wrong; it does not depend on foundation but just dispatch hm.

https://github.com/apple/swift-metrics/blob/44e8bfc7f5a0686e21d5d21a7ea5454e74ed3449/Sources/Metrics/Metrics.swift#L42-L44

FYI @tomerd @slashmo

ktoso avatar Mar 18 '21 04:03 ktoso

I think depending on Dispatch in Core is fine, there are a couple of other ones in there that can move too

tomerd avatar Mar 18 '21 18:03 tomerd