cldr icon indicating copy to clipboard operation
cldr copied to clipboard

CLDR-16503 test for consistent English units

Open srl295 opened this issue 2 years ago • 6 comments

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

srl295 avatar May 12 '23 18:05 srl295

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.

macchiati avatar May 13 '23 00:05 macchiati

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.

macchiati avatar May 13 '23 00:05 macchiati

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.

srl295 avatar May 13 '23 00:05 srl295

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: @.***>

macchiati avatar May 13 '23 00:05 macchiati

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?

srl295 avatar Jun 05 '23 16:06 srl295