Perfect, I'll merge this when the PR on InfiniTime gets merged. Thank you.
infinitime.go, line 722:
I think silent failure would be better in this case, just because many people won't have 1.10.0 at first, and printing warnings every time would be pretty annoying for…
Your version will return the timezone offset at Now(). You see, the timezone offset is not constant over time (see DST). That is why the timezone usually is tied to a datetime to make it…
Ok, I'll just copy them here.
infinitime.go, line 720:
Overall, this PR looks good, but this seems like it should accept a *time.Location instead of time.Time because that would be a bit…
This looks good. Please look at the comment I left on Arsen6331/infinitime#4. I will merge once the InfiniTime PR gets merged.
Thank you for this PR. I don't want to merge this until the InfiniTime PR gets merged, so I will keep this open for now. In the meantime, please apply my suggestion in the comment I wrote…