search

NorfairKing / feedback

Declarative feedback loop manager
https://cs-syd.eu/posts/2022-11-14-automate-your-feedback-loops-using-feedback
GNU General Public License v3.0
61 stars 10 forks source link

Timestamps do not account for time zone #10

Closed Bodigrim closed 2 years ago

Bodigrim commented 2 years ago

https://github.com/NorfairKing/feedback/blob/3928be61d28d480d043eb01394c8e55478ab969e/feedback/src/Feedback/Common/Output.hs#L20-L23

I've been a bit confused by timestamps, referring far in the past. The thing is that getCurrentTime returns UTCTime and does not account for local time zone. I'm not sure, but maybe getZonedTime is a better default?

NorfairKing commented 2 years ago

Oh, duh! That's a silly mistake :p

NorfairKing commented 2 years ago

Quickly fixed!

Bodigrim commented 2 years ago

Thanks!