-
-
Notifications
You must be signed in to change notification settings - Fork 42
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
Address confusable AIs for 16.0 #841
Address confusable AIs for 16.0 #841
Conversation
macchiati
commented
May 31, 2024
•
edited
Loading
edited
Action | Description | Documents |
---|---|---|
176-A116 | Update confusable data for UTS #39 considering U+A7DA LATIN CAPITAL LETTER LAMBDA, U+A7DB LATIN SMALL LETTER LAMBDA, U+A7DC LATIN CAPITAL LETTER LAMBDA WITH STROKE. (Reference: Section 2a of L2/23-164) | L2/23-164 |
178-A76 | Add U+16FF2 CHINESE SMALL SIMPLIFIED ER and U+16FF3 CHINESE SMALL TRADITIONAL ER to the confusables list in UTS #39 as per Section 21 of document L2/24-012. | L2/24-012 |
166-A55 | For Unicode 14 confusables.txt, add ß ~ β and ẞ ~ B if feasible. And Middle Scots S. See document L2/21-016 (U+A7D6 LATIN CAPITAL LETTER MIDDLE SCOTS S, U+A7D7 LATIN SMALL LETTER MIDDLE SCOTS S) (Retargeted to 15.0.) | L2/21-016 |
165-A37 | Add U+1715 TAGALOG SIGN PAMUDPOD to the list of confusables. | L2/20-272 |
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.
FYI it looks like this addresses two action items so far. I added those two to the PR description and set them to "In progress" in the UTC AI spreadsheet.
unicodetools/data/security/dev/data/source/confusables-source.txt
Outdated
Show resolved
Hide resolved
unicodetools/data/security/dev/data/source/confusables-source.txt
Outdated
Show resolved
Hide resolved
unicodetools/data/security/dev/data/source/confusables-source.txt
Outdated
Show resolved
Hide resolved
Note: There are some older submissions from David Corbett that look good. I didn't want to combined into this PR, because it is easier to manage smaller chunks. |
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.
please take a look at my feedback
Thanks |
Hi @macchiati sorry for approving only last night... Is this ready to merge? Can you please squash-and-merge, or if you don't have time, would you mind if I did? |
Merging is great (he said after the fact)
…On Tue, Jun 4, 2024, 19:07 Markus Scherer ***@***.***> wrote:
Merged #841 <#841> into
main.
—
Reply to this email directly, view it on GitHub
<#841 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACJLEMFNGDWJVKHLJZBE65DZFZXG5AVCNFSM6AAAAABISBQDHSVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGA2DIMJSGQYDGOA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|