cldr icon indicating copy to clipboard operation
cldr copied to clipboard

CLDR-15698 Time examples < 10 and > 12

Open e-ikwut opened this issue 1 year ago • 11 comments

output.txt

before: image

after: image

CLDR-15698

  • [CLDR-15698 ] This PR completes the ticket.

ALLOW_MANY_COMMITS=true

e-ikwut avatar Jul 23 '24 17:07 e-ikwut

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Jul 23 '24 17:07 CLAassistant

The output.txt is useful, but best to also have before and after screen shots also. And for the text output, also best to have the before and after.

macchiati avatar Jul 25 '24 18:07 macchiati

There is a commit message that fails validation. The message must be of the form:

CLDR-15698

eg

CLDR-15698 Time examples < 10 and > 12

The "Details" button on the "jira-ticket — Commit message for 41fe3cb fails validation" gives you a way to fix that, by squashing. Or you can make a new PR. Let us know on slack if you have trouble with that.

macchiati avatar Jul 26 '24 00:07 macchiati

Hooray! The files in the branch are the same across the force-push. 😃

~ Your Friendly Jira-GitHub PR Checker Bot

The screenshots look good.

macchiati avatar Jul 29 '24 01:07 macchiati

Will the same examples show in the Info Panel?

AEApple avatar Jul 31 '24 00:07 AEApple

Will the same examples show in the Info Panel?

Yes, the same set shows in the pop-up and the info panel.

e-ikwut avatar Jul 31 '24 13:07 e-ikwut

Once you are ready for review, change the status to "Ready for Review". I approved, but you'll have to fix the conflicts and ask for a re-review.

macchiati avatar Aug 19 '24 23:08 macchiati

Notice: the branch changed across the force-push!

  • tools/cldr-code/src/main/java/org/unicode/cldr/test/ExampleGenerator.java is no longer changed in the branch

View Diff Across Force-Push

~ Your Friendly Jira-GitHub PR Checker Bot

Hooray! The files in the branch are the same across the force-push. 😃

~ Your Friendly Jira-GitHub PR Checker Bot

Hooray! The files in the branch are the same across the force-push. 😃

~ Your Friendly Jira-GitHub PR Checker Bot

@e-ikwut needs mvn spotless:apply can you do that? make sure the commit is formatted properly

srl295 avatar Sep 03 '24 18:09 srl295