CLDR-16503 test for consistent English units
CLDR-16503
- [ ] This PR completes the ticket.
Currently fails,
Inconsistent units, missing from en: volume-dessert-spoon-imperial(short) volume-quart-imperial(short) volume-dessert-spoon(short) volume-pinch(short) light-candela(short) mass-grain(short) light-lumen(short) volume-drop(short) volume-dram(short) temperature-generic(narrow, short) volume-jigger(short) length-earth-radius(short) graphics-dot(short)
ALLOW_MANY_COMMITS=true
We haven't guaranteed that English has values for all root values, so I think that might be what you are seeing. But that is a broader issue, not just units.
Also there is already a tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestUnits.java (different directory). We shouldn't add duplicate names even in different directories.
Also there is already a tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestUnits.java (different directory). We shouldn't add duplicate names even in different directories.
it's the non-JUnit version.
- [ ] I can disambiguate the name.
Disambiguating the name sounds good. But should they be in the same directory, or is that because of our historical structure didn't match?
On Fri, May 12, 2023 at 5:22 PM Steven R. Loomis @.***> wrote:
Also there is already a tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestUnits.java (different directory). We shouldn't add duplicate names even in different directories.
it's the non-JUnit version.
- I can disambiguate the name.
— Reply to this email directly, view it on GitHub https://github.com/unicode-org/cldr/pull/2921#issuecomment-1546463230, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACJLEMAZRHLC5P6WNG3GVJDXF3H4LANCNFSM6AAAAAAX72ZQEQ . You are receiving this because you commented.Message ID: @.***>
Hooray! The files in the branch are the same across the force-push. 😃
~ Your Friendly Jira-GitHub PR Checker Bot
this ticket is closing, so if desired this should move to a new ticket?