-
-
Notifications
You must be signed in to change notification settings - Fork 439
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add the timezone in clock meter #1459
base: main
Are you sure you want to change the base?
Conversation
I would at least make this optional (config option) as (... how do I say that politely ...) it is not a sign of following best practices to run distributed servers with clocks in local time zones. |
Nah. It's about the display of the system time in UTC or in local time. Note the colck meter uses The switch between local time and UTC can be done as a separate feature. |
Whether or not the timezone being local on a server makes sense... Depends. But that's the point: My workstation has local timezone Will have a look to make it configurable. |
I am confused how sysadmins would be confused about the |
Bring your |
That's all the code that would be affected by time zones. And yes, if you show the timezone offset in ClockMeter, it should also show in DateTimeMeter. |
Well, my confusion was not about the settings of the server, but more about what terminal window I was looking at. 😜
Because that's what I use. 😁 Added it to date/time meter in another commit, but still without any configuration option.
I would have to add that in user configuration of any server... Using environment variables via |
915a6c1
to
4b04865
Compare
I connect to servers with non-local timezones every now and then... Let's add the timezone in clock meter to minimize confusion there.
... just as done in clock meter.
4b04865
to
ea376a9
Compare
I connect to servers with non-local timezones every now and then... Let's add the timezone in clock meter to minimize confusion there.