-
Notifications
You must be signed in to change notification settings - Fork 386
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-17765 BRS CLDRModify common/annotations; no -fQ; fix ka.xml #3992
Conversation
-Hand-edit annotations/ka.xml to remove 6 / 5000 Результаты перевода -Disable -fQ in CLDRModify.java -Run CLDRModify on common/annotations without options
@@ -3416,7 +3416,7 @@ Warnings: All cp values have U+FE0F characters removed. See /annotationsDerived/ | |||
<annotation cp="🧿">ავგაროზი | თილისმა | თურქული თვალი | ნაზარი | ფატიმას თვალი</annotation> | |||
<annotation cp="🧿" type="tts">ფატიმას თვალი</annotation> | |||
<annotation cp="🪬">ამულეტი | დაცვა | მარიამი | მირიამი | ფატიმა | ხელი | ჰამსა</annotation> | |||
<annotation cp="🪬" type="tts">6 / 5000 Результаты перевода ჰამსა</annotation> | |||
<annotation cp="🪬" type="tts">ჰამსა</annotation> |
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.
This is hand-edited
@@ -650,11 +650,11 @@ private static void checkSuboptions(int i, UnicodeSet allowedOptions) { | |||
+ "Use -? for help."); | |||
} | |||
if (i == FIX && givenOptions.value != null) { | |||
final UnicodeSet allowedFilters = new UnicodeSet().add('P').add('Q').add('k'); | |||
final UnicodeSet allowedFilters = new UnicodeSet().add('P').add('k'); |
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.
This is the tooling change to disable -fQ
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.
Thanks!
@@ -650,11 +650,11 @@ private static void checkSuboptions(int i, UnicodeSet allowedOptions) { | |||
+ "Use -? for help."); | |||
} | |||
if (i == FIX && givenOptions.value != null) { | |||
final UnicodeSet allowedFilters = new UnicodeSet().add('P').add('Q').add('k'); | |||
final UnicodeSet allowedFilters = new UnicodeSet().add('P').add('k'); |
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.
Thanks!
-Hand-edit annotations/ka.xml to remove 6 / 5000 Результаты перевода
-Disable -fQ in CLDRModify.java
-Run CLDRModify on common/annotations without options
CLDR-17765
ALLOW_MANY_COMMITS=true