Skip to content

CLDR-19122 add preferred zones#5193

Draft
robertbastian wants to merge 1 commit intounicode-org:mainfrom
robertbastian:preferred
Draft

CLDR-19122 add preferred zones#5193
robertbastian wants to merge 1 commit intounicode-org:mainfrom
robertbastian:preferred

Conversation

@robertbastian
Copy link
Copy Markdown
Member

CLDR-19122

  • This PR completes the ticket.

ALLOW_MANY_COMMITS=true

@macchiati
Copy link
Copy Markdown
Member

Looking good so far.

@robertbastian
Copy link
Copy Markdown
Member Author

I compiled this by generating a list of time zone -> metazone golden mappings for the same country, but as those also include zones that differ by DST, I manually checked all of them. It might be possible to make this fully algorithmic, but it would be a non-trivial algorithm, so I think it makes sense to hardcode them (and we can test against metazones if we want).

@macchiati
Copy link
Copy Markdown
Member

I compiled this by generating a list of time zone -> metazone golden mappings for the same country, but as those also include zones that differ by DST, I manually checked all of them. It might be possible to make this fully algorithmic, but it would be a non-trivial algorithm, so I think it makes sense to hardcode them (and we can test against metazones if we want).

I think it would be a good idea to have tests for this; while there is some latitude, there should be some upper and lower bounds that can be tested for (and we'll have access to the tzdb)

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