-
Notifications
You must be signed in to change notification settings - Fork 182
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
CLDR 43 Beta 3 #3185
CLDR 43 Beta 3 #3185
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Praise: Thank you very much for keeping code changes and data changes in their own separate PRs
@@ -1,18 +1,59 @@ | |||
{ | |||
"l2s": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thought/Issue, not your fault: we should filter this list so that it doesn't grow so much on this update. I worked out a solution for the main likelysubtags but not this fallback side version. Actually I would like to deprecate this key entirely and we can do that now.
Let's get this merged please |
Please land #3225 first. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
^
@@ -751,9 +750,8 @@ displaynames/regions@1, sr-Cyrl, 7715B, 51420e0d3503da27 | |||
displaynames/regions@1, sr-Latn, 4776B, e5961905fd208c83 | |||
displaynames/regions@1, th, 10811B, 84aad4d11372eb38 | |||
displaynames/regions@1, tr, 4650B, 8f8bd5f16f02a43 | |||
displaynames/regions@1, und, 2138B, 87850fe8d7efdd55 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Question: What happened to displaynames/regions@1, und
and also displaynames/languages@1, und
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
und
only contained fallback mappings (DE
-> DE
), which are removed in 43.
#3181