-
Notifications
You must be signed in to change notification settings - Fork 387
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-17416 v45 BRS validity #3531
Conversation
- also starting to pickup spdx changes from CLDR-17110
Tests failing |
@@ -35,12 +35,12 @@ | |||
Vaii Vith | |||
Wara Wcho | |||
Xpeo Xsux | |||
Yezi Yiii Zanb |
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.
what's going on with Zanb
? @markusicu @macchiati
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.
We've been here before.. #3233 (comment)
blocked by CLDR-17056 - I'll back out Zanb
for now.
- and add comment linking CLDR-17056 in the xml
@@ -35,7 +35,7 @@ | |||
Vaii Vith | |||
Wara Wcho | |||
Xpeo Xsux | |||
Yezi Yiii Zanb | |||
Yezi Yiii Zanb <!-- Note: for Zanb see CLDR-17056 --> |
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.
^ so that we see this the next time the generator is run
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.
I decided it was time to fix the tooling. I figured that rather than have the tooling be a separate PR, it was better to do it all at once. So could you review #3537 and if that's ok, merge it and close this PR?
CLDR-17416
ALLOW_MANY_COMMITS=true