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

Use the canonical timezone for Mountain Time #71

Merged
merged 1 commit into from
Nov 10, 2024

Conversation

stefanor
Copy link
Contributor

The main tzdata database only contains the canonical names, abbreviations may not be installed. In Debian they're packaged in the tzdata-legacy package.

The main tzdata database only contains the canonical names,
abbreviations may not be installed. In Debian they're packaged in the
tzdata-legacy package.
@CalebBell CalebBell merged commit 0cf78a4 into CalebBell:master Nov 10, 2024
70 checks passed
@CalebBell
Copy link
Owner

Hi Stefano,
Thank you very kindly for this fix. I have released a new version of fluids 1.0.27, with the fix.
I recently added some CI to run the tests of fluids on other CPU architectures through QEMU specifically because of another time some tests had failed in debian with minor numerical differences.
I will continue doing my best to keep the tests in fluids high quality and reliable. Please do not hesitate to reach out if there are any issues in the future.
Sincerely,
Caleb

@stefanor stefanor deleted the canonical-tz branch November 10, 2024 21:45
@stefanor
Copy link
Contributor Author

Thank you very kindly for this fix. I have released a new version of fluids 1.0.27, with the fix.

Thank you.

It looks like it just missed the tag, but I see it in the changelog for 1.0.27.

@CalebBell
Copy link
Owner

Thanks for catching that! Yes it was a bit odd getting this PR at the same time as I was in the middle of releasing. I've corrected which commit had the tag to the latest in master, with the changelog update commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants