Skip to content

[UTC-181-A132] update CJKRadicals.txt #1135

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

Merged
merged 2 commits into from
May 7, 2025
Merged

[UTC-181-A132] update CJKRadicals.txt #1135

merged 2 commits into from
May 7, 2025

Conversation

josh-hadley
Copy link
Collaborator

Update the description of CJK radical numbers in CJKRadicals.txt to be consistent with the use of apostrophes per the kRSUnicode property. For Unicode Version 17.0. See L2/24-224 item 2.5.

Update the description of CJK radical numbers in CJKRadicals.txt to be consistent with the use of apostrophes per the kRSUnicode property. For Unicode Version 17.0. See L2/24-224 item 2.5.
@josh-hadley josh-hadley requested a review from markusicu May 7, 2025 04:24
@josh-hadley josh-hadley changed the title [UTC-183-A132] update CJKRadicals.txt [UTC-181-A132] update CJKRadicals.txt May 7, 2025
@josh-hadley
Copy link
Collaborator Author

@markusicu if you ✅ this you can go ahead and merge

Copy link
Member

@markusicu markusicu left a comment

Choose a reason for hiding this comment

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

Actually, please change lines 2 & 3 of this file to

# Date: 2025-05-07
# © 2025 Unicode®, Inc.

@josh-hadley
Copy link
Collaborator Author

Actually, please change lines 2 & 3 of this file

Done, thanks for catching that 😸

Copy link
Member

@markusicu markusicu left a comment

Choose a reason for hiding this comment

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

tnx pse squash-and-merge

@markusicu markusicu merged commit 711cae6 into main May 7, 2025
23 checks passed
@markusicu markusicu deleted the jh-cjkradicals branch May 7, 2025 17:51
@markusicu
Copy link
Member

@markusicu if you ✅ this you can go ahead and merge

done, thanks for this notice :-)

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