Skip to content

chore(resources): CLDR 49 - #16359

Draft
srl295 wants to merge 7 commits into
keymanapp:masterfrom
srl295:chore/resources/cldr-49
Draft

chore(resources): CLDR 49#16359
srl295 wants to merge 7 commits into
keymanapp:masterfrom
srl295:chore/resources/cldr-49

Conversation

@srl295

@srl295 srl295 commented Aug 13, 2026

Copy link
Copy Markdown
Member

Test-bot: skip

srl295 added 2 commits August 12, 2026 20:07
- treat 45…49, compatible DTD
- debug for unsupported stuff
- includes WIP keyboard files that sort of compile
@github-project-automation github-project-automation Bot moved this to Todo in Keyman Aug 13, 2026
@keymanapp-test-bot keymanapp-test-bot Bot added the user-test-missing User tests have not yet been defined for the PR label Aug 13, 2026
@keymanapp-test-bot

keymanapp-test-bot Bot commented Aug 13, 2026

Copy link
Copy Markdown

User Test Results

Test specification and instructions

User tests are not required

@keymanapp-test-bot keymanapp-test-bot Bot added this to the A19S35 milestone Aug 13, 2026
@keymanapp-test-bot keymanapp-test-bot Bot removed the user-test-missing User tests have not yet been defined for the PR label Aug 13, 2026
@keyman-server

Copy link
Copy Markdown
Collaborator

This pull request is from an external repo and will not automatically be built. The build must still be passed before it can be merged. Ask one of the team members to make a manual build of this PR.

@mcdurdin mcdurdin changed the title chore: CLDR 49 chore(resources): CLDR 49 Aug 13, 2026
@mcdurdin

Copy link
Copy Markdown
Member

😕 srl295 reacted with confused emoji

Because you created this PR from a branch on your fork, not on this repo.

@srl295

srl295 commented Aug 13, 2026

Copy link
Copy Markdown
Member Author

😕 srl295 reacted with confused emoji

Because you created this PR from a branch on your fork, not on this repo.

yeah… needs a proper issue etc..

@mcdurdin

Copy link
Copy Markdown
Member

We'll need to split this into the CLDR 49 update and separate bug patches I think

@srl295

srl295 commented Aug 14, 2026 via email

Copy link
Copy Markdown
Member Author

@mcdurdin mcdurdin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do you have stacks ?Any particular branch it should be on?

Stacks? Not sure what you mean there.

It looks like there are three different things:

  1. ICU 73->77
  2. LDML 46->49 (including the additional draft and attribution props)
  3. kmx_xstring casts

Is there a reason to keep ICU and LDML in the same PR? Otherwise, for separation of concerns, I'd split those updates as well.

layout?: string;
indicator?: string;
attribution?: string;
draft?: string;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
draft?: string;

draft is on keyboard3, not on info?

@srl295

srl295 commented Aug 14, 2026

Copy link
Copy Markdown
Member Author

Didn't build for me without the icu and kmx fixes. But I didn't read the instructions !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

3 participants