Skip to content
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

CLDR-17843 Update xnr.xml -- Add scaffolding for Kangri Devanagari nu… #3921

Conversation

conradarcturus
Copy link
Contributor

@conradarcturus conradarcturus commented Jul 31, 2024

…mber formatting patterns

We don't have submitted data for the Devanagari numerals in the Kangri language -- so we'll fall back to what Hindi is doing, which is just to inherit parent values atm.

This will prevent Kangri from dropping in coverage level.

CLDR-17843

  • This PR completes 3/7 locales mentioned in the ticket.

ALLOW_MANY_COMMITS=true
DISABLE_JIRA_ISSUE_MATCH=true

@srl295
Copy link
Member

srl295 commented Jul 31, 2024

remember we'll need to regenerate coverageLevels.txt after this merges as in #3897

conradarcturus and others added 3 commits August 3, 2024 11:44
…mber formatting patterns

We don't have submitted data for the Devanagari numerals in the Kangri language -- so we'll fall back to what Hindi is doing, which is just to inherit parent values atm.

This will prevent Kangri from dropping in coverage level.
…agari digits

In order to prevent Kuvi from dropping in coverage, this updates the number formats to inherit from root, otherwise it will be considered to be missing information.
…ols inherit same format as root

Adding N'Ko update to the PR.

In order to prevent N'Ko from dropping in coverage, this updates number formats for their native digits to re-use the patterns from Latin -- while there may be differences we will request confirmation from linguists later.
common/main/kxv.xml Outdated Show resolved Hide resolved
srl295
srl295 previously requested changes Aug 5, 2024
Copy link
Member

@srl295 srl295 left a comment

Choose a reason for hiding this comment

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

needs a fix

@conradarcturus conradarcturus force-pushed the CLDR-17843-Fix-locales-about-to-drop-in-coverage-1 branch from ec094f4 to 3307be2 Compare August 5, 2024 18:20
@conradarcturus
Copy link
Contributor Author

Fix applied, please re-review

@conradarcturus conradarcturus requested a review from srl295 August 6, 2024 17:58
The child locales of Kuvi needed inheritance as well so I fixed that too.

Also re-ran the coverage levels `java -jar tools/cldr-code/target/cldr-code.jar ShowLocaleCoverage`
@conradarcturus conradarcturus force-pushed the CLDR-17843-Fix-locales-about-to-drop-in-coverage-1 branch from cca247c to b2dc0c8 Compare August 6, 2024 18:16
@jira-pull-request-webhook
Copy link

Notice: the branch changed across the force-push!

  • common/main/kxv_Deva.xml is now changed in the branch
  • common/main/kxv_Orya.xml is now changed in the branch
  • common/main/kxv_Telu.xml is now changed in the branch
  • common/main/kxv.xml is different
  • common/properties/coverageLevels.txt is now changed in the branch

View Diff Across Force-Push

~ Your Friendly Jira-GitHub PR Checker Bot

@conradarcturus conradarcturus dismissed srl295’s stale review August 6, 2024 22:25

Changes accepted.

@conradarcturus conradarcturus merged commit 740cf77 into unicode-org:main Aug 7, 2024
9 checks passed
@conradarcturus conradarcturus deleted the CLDR-17843-Fix-locales-about-to-drop-in-coverage-1 branch August 7, 2024 15:52
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