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
The time zone "Europe/Amsterdam" is of class TimeZones.Class(:LEGACY)
#419
Comments
( The messages on this page were enlightening to me: https://mm.icann.org/pipermail/tz/2022-July/031642.html So if TimeZones.jl wants to mirror IANA, then perhaps this is correct behaviour. ) From looking at the code, it seems that we're not currently looking at the There is this comment in the file
|
Sorry about this disruptive change. Typically the tzdata updates are rather minor but this latest one changed more than I realized. I'll try to add some tests to catch these more major changes and possibly tag these changes as a major release (need to give this a little more thought). Alternatively, we may want to add support for properly deprecating these kinds of class changes so that Julia code doesn't break with tzdata changes. You can still construct these time zones using
The |
Is there any functional difference between |
I'm unable to run code which refers to
tz"Europe/Amsterdam"
withTimeZones.jl
v1.9.1:This is unexpected since
Europe/Amsterdam
is a commonly used timezone. The release notes of tz show no mention of the deprecation of the timezone, apart from the following note:source: https://github.com/eggert/tz/blob/16ce126a87c5f130cde8b8dce73b38952a19f085/NEWS#L300-L312
And indeed the same error shows up for the zones listed in the note. For example:
Is this behavior expected?
The text was updated successfully, but these errors were encountered: