Skip to content

fix: remove timex package#152

Merged
nrwiersma merged 1 commit intomainfrom
remove-timex
Feb 14, 2025
Merged

fix: remove timex package#152
nrwiersma merged 1 commit intomainfrom
remove-timex

Conversation

@nrwiersma
Copy link
Member

Goal of this PR

This removes the use of github.com/hamba/timex as it no longer has a speed advantage.

name         old time/op    new time/op    delta
WithStats-8     164ns ±20%     158ns ±14%   ~     (p=0.529 n=10+10)

name         old alloc/op   new alloc/op   delta
WithStats-8     41.0B ± 0%     41.0B ± 0%   ~     (all equal)

name         old allocs/op  new allocs/op  delta
WithStats-8      4.00 ± 0%      4.00 ± 0%   ~     (all equal)

How did I test it?

@nrwiersma nrwiersma self-assigned this Feb 14, 2025
@nrwiersma nrwiersma merged commit a88a708 into main Feb 14, 2025
2 checks passed
@nrwiersma nrwiersma deleted the remove-timex branch February 14, 2025 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant