-
Notifications
You must be signed in to change notification settings - Fork 399
CLDR-18920 Update modify_config.txt for en_001 #5013
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
base: main
Are you sure you want to change the base?
Conversation
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. |
Ok, can do.
Setting up your IDE is not something you want to do under pressure!
…On Wed, Sep 3, 2025 at 7:37 PM Annemarie Apple ***@***.***> wrote:
*AEApple* left a comment (unicode-org/cldr#5013)
<#5013 (comment)>
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.....
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.
—
Reply to this email directly, view it on GitHub
<#5013 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACJLEMHTQHZDH45HXDPIAF33Q6QYJAVCNFSM6AAAAACFSEUJW2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTENJRGUZTCMZQHA>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Makes sense, but I will add instructions to our docs once I figure out how to set up a cloud workspace. |
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. |
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.
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> |
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.
@macchiati - I think I probably shouldn't have changed this one. Interestingly how both en and en-GB have the longer forms. WDYT?
Hooray! The files in the branch are the same across the force-push. 😃 ~ Your Friendly Jira-GitHub PR Checker Bot |
CLDR-18920
ALLOW_MANY_COMMITS=true