print times in local timezone #217

Merged
6543 merged 5 commits from noerw/tea:local-time into master 2 years ago
noerw commented 2 years ago
Collaborator
  • before, times would be shown in UTC. now it is shown in local timezone. if a timezone can't be determined, it still falls back to UTC, but is annotated as such. timezone can be overriden with the TZ env variable.

  • PR also reduces precision of printed times to minutes instead of seconds.

- before, times would be shown in UTC. now it is shown in local timezone. if a timezone can't be determined, it still falls back to UTC, but is annotated as such. timezone can be overriden with the `TZ` env variable. - PR also reduces precision of printed times to minutes instead of seconds.
noerw added 1 commit 2 years ago
continuous-integration/drone/pr Build is passing Details
fab5ad1762
use local timezone for all printed times
noerw added the status/needs-reviews kind/enhancement labels 2 years ago
noerw added 1 commit 2 years ago
continuous-integration/drone/pr Build is passing Details
02edd6aa24
fixup! use local timezone for all printed times
noerw added 1 commit 2 years ago
continuous-integration/drone/pr Build is passing Details
82d4fa4598
fixup! use local timezone for all printed times
6543 approved these changes 2 years ago
Dismissed
6543 left a comment
Collaborator

I'm not a fan of init but I guess it's faster this way

I'm not a fan of `init` but I guess it's faster this way
Poster
Collaborator

I just looked at the internals of the time package, and the local zone is looked up only once, so we can get the location everytime without issues

I just looked at the internals of the `time` package, and the local zone is looked up only once, so we can get the location everytime without issues
noerw added 1 commit 2 years ago
continuous-integration/drone/pr Build is passing Details
16d1531f05
dont resolve location ahead of time
noerw changed title from use local timezone for all times to print times in local timezone 2 years ago
techknowlogick approved these changes 2 years ago
Dismissed
techknowlogick left a comment
Owner

LGTM, please resolve conflicts :)

LGTM, please resolve conflicts :)
techknowlogick removed the status/needs-reviews label 2 years ago
noerw added 1 commit 2 years ago
continuous-integration/drone/pr Build is passing Details
46189e9c83
Merge branch 'master' into local-time
Poster
Collaborator

@techknowlogick done :>

@techknowlogick done :>
noerw added this to the v0.6.0 milestone 2 years ago
6543 approved these changes 2 years ago
Dismissed
6543 merged commit 03ec6d0eee into master 2 years ago
6543 referenced this issue from a commit 2 years ago
noerw deleted branch local-time 2 years ago
continuous-integration/drone/pr Build is passing
The pull request has been merged as 03ec6d0eee.
Sign in to join this conversation.
Loading…
There is no content yet.