Skip to content
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

Fix BIT + Etc/ timezones #198

Merged
merged 2 commits into from
Jan 13, 2023
Merged

Fix BIT + Etc/ timezones #198

merged 2 commits into from
Jan 13, 2023

Conversation

Zephyrkul
Copy link
Contributor

According to the adjacent comment (and Wikipedia), Baker Island is UTC−12. However, the timezone passed is incorrectly set to Pacific/Funafuti, a timezone of Tuvalu, which is on the other side of the International Date Line from Baker Island (and thus UTC+12). This PR sets BIT to Etc/GMT+12, which is UTC−12.

The other Etc/ timezones have also been fixed, as the signs for the Etc/ area are flipped from ISO conventions. ([cite 1], [cite 2])

@Zephyrkul
Copy link
Contributor Author

black is failing on this PR but this is due to files in other areas of the repo (ccrole/ccrole.py and fifo/task.py, specifically) - the only file this PR touches would not be reformatted.

@bobloy bobloy merged commit 5f99f3d into bobloy:master Jan 13, 2023
@Zephyrkul Zephyrkul deleted the patch-1 branch January 14, 2023 03:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants