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

Initial transition effective from typemin(DateTime) #10

Merged
merged 1 commit into from
Oct 13, 2015
Merged

Conversation

omus
Copy link
Member

@omus omus commented Oct 13, 2015

VariableTimeZone types are made up of transitions which have a UTC DateTime indicating the instant at when the new rule becomes effective. Previously the first transition started at 1800/1/1 for no real reason except that no zone or rule started that early and it is rather silly to use the concept of time zones prior to their use. After merging in #8 it seems more reasonable to allow users to make ZonedDateTime's prior to 1800 and let them use their own judgement.

As a happy side effect this makes comparing Julia processed timezone rules easier to compare with tzfiles.

@quinnj
Copy link
Collaborator

quinnj commented Oct 13, 2015

+1

omus added a commit that referenced this pull request Oct 13, 2015
Initial transition effective from typemin(DateTime)
@omus omus merged commit 4390d01 into master Oct 13, 2015
@omus omus deleted the olson_min branch October 13, 2015 17:16
omus pushed a commit to invenia/TimeZones.jl that referenced this pull request Aug 15, 2018
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