semantic-kernel icon indicating copy to clipboard operation
semantic-kernel copied to clipboard

Adopt TimeProvider abstraction when its ready

Open stephentoub opened this issue 2 years ago • 1 comments

The DefaultHttpRetryHandler type defines IDelayProvider and ITimeProvider interfaces. The APIs these surface are fully covered by the new System.TimeProvider abstract type being introduced in .NET 8 but that will also be available from the Microsoft.Bcl.TimeProvider nuget package downlevel. Once it's available, it should ideally be used instead of Semantic Kernel defining its own abstractions.

stephentoub avatar Apr 24 '23 03:04 stephentoub

Agreed, once we have a chance to revisit the unit tests approach, we'll start using this, removing the bespoke version

dluc avatar Apr 26 '23 16:04 dluc