temporal icon indicating copy to clipboard operation
temporal copied to clipboard

Investigate using `EpochNanoseconds` and implement wherever possible

Open nekevss opened this issue 1 year ago • 1 comments

In #116, the EpochNanoseconds new type was added. The introduction of EpochNanoseconds was fairly bare, as further mentioned in #115. We should look into using EpochNanoseconds more throughout the codebase where it makes sense according to the proposal.

nekevss avatar Dec 05 '24 03:12 nekevss

I would like to try this one!

JohannesHelleve avatar Apr 09 '25 10:04 JohannesHelleve

This is more or less complete, and no longer planned.

The general conclusion:

The TimeZoneProvider trait is be redesigned and move out of temporal_rs, so it will not have access to EpochNanoseconds. Furthermore, there are intermediate points in calculations where an EpochNanosecond could actually exceed the allowed range.

nekevss avatar Aug 26 '25 20:08 nekevss