Skip to content

Conversation

AEApple
Copy link
Contributor

@AEApple AEApple commented Sep 3, 2025

CLDR-18920

  • This PR completes the ticket.

ALLOW_MANY_COMMITS=true

@macchiati
Copy link
Member

What we would normally do is run the CLDRModify tool with -k, as part of the same PR. I'm guessing that you don't have access to an IDE to do that; if so, I can jump in and do it.

BTW, I've been wondering whether we could run tools with github workspace; that might be worth investigating!

@AEApple
Copy link
Contributor Author

AEApple commented Sep 4, 2025

What we would normally do is run the CLDRModify tool with -k, as part of the same PR. I'm guessing that you don't have access to an IDE to do that; if so, I can jump in and do it.

BTW, I've been wondering whether we could run tools with github workspace; that might be worth investigating!

I was going to reset up my IDE but I've been trying to figure out where to edit JAVA_HOME in the GitHub Codespace, but I haven't found the right answer yet..... I was planning to switch to a local IDE if I don't find the answer soon, but I'm unfortunately much more familiar with the internal tooling than GitHub.

It may be faster if you do this, and I'll keep digging around for the answer. I want to figure out how to set up and use the Codespaces as for my IDE. It SEEMS like it should be possible.

@AEApple AEApple changed the title Update modify_config.txt for en_001 CLDR-18920 Update modify_config.txt for en_001 Sep 4, 2025
@macchiati
Copy link
Member

macchiati commented Sep 4, 2025 via email

@AEApple
Copy link
Contributor Author

AEApple commented Sep 4, 2025

Ok, can do. Setting up your IDE is not something you want to do under pressure!

Makes sense, but I will add instructions to our docs once I figure out how to set up a cloud workspace.

@macchiati
Copy link
Member

I generated the two files based on the modify_config. Please spot check them for completeness.

NOTE: I should have remembered that currently we need to modify the annotations and main directories separated, so I ended up doing two passes, modifying the modify_config for each.

Copy link
Member

@macchiati macchiati left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stamping so that you can merge once you're reviewed my changes.

@@ -435,7 +437,7 @@ Warnings: All cp values have U+FE0F characters removed. See /annotationsDerived/
<greatestDifference id="M">d MMM – d MMM</greatestDifference>
</intervalFormatItem>
<intervalFormatItem id="MMMEd">
<greatestDifference id="d">E d – E d MMM</greatestDifference>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@macchiati - I think I probably shouldn't have changed this one. Interestingly how both en and en-GB have the longer forms. WDYT?

@AEApple AEApple requested a review from stenshamn September 5, 2025 04:28
@jira-pull-request-webhook
Copy link

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

~ Your Friendly Jira-GitHub PR Checker Bot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants