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

chore: merge master into shared-fonts 🐻 #12782

Merged
merged 19 commits into from
Dec 5, 2024

Conversation

keyman-server
Copy link
Collaborator

ermshiperete and others added 18 commits November 28, 2024 15:28
This change rename the test files in Common according to the discussion
at the Keyman conference in November 2024.
- common: shift the 'strs' processing slightly, because otherwise we normalize to NFD before even tracking the strings
- if any string is neither NFC nor NFD, give a warning
- add tests for the same

Fixes: #7394
… review comments

- report a warning on each separate instance of a denormalized string (once per string).

Fixes: #7394
…hint-for-non-matching-script

fix(developer): honour provided script when checking for matching scripts
This change renames a new test file and moves it to the new location.
This file was overlooked in #12709.
auto: increment master version to 18.0.153
…ormalization-epic-ldml

feat(developer,common): verify normalization of strings 🙀
chore(core): Add link to Keyman Glossary 🎄
@github-actions github-actions bot added the chore label Dec 5, 2024
@keymanapp-test-bot keymanapp-test-bot bot changed the title chore: merge master into shared-fonts chore: merge master into shared-fonts 🐻 Dec 5, 2024
@keymanapp-test-bot keymanapp-test-bot bot added this to the A18S16 milestone Dec 5, 2024
@mcdurdin mcdurdin merged commit c150483 into epic/shared-fonts Dec 5, 2024
28 checks passed
@mcdurdin mcdurdin deleted the chore/merge-master-into-shared-fonts branch December 5, 2024 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants