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

Upgrade to Unicode 16.0 #7

Merged
merged 3 commits into from
Oct 8, 2024
Merged

Upgrade to Unicode 16.0 #7

merged 3 commits into from
Oct 8, 2024

Conversation

wezm
Copy link

@wezm wezm commented Oct 3, 2024

No description provided.

Copy link
Member

@adrianwong adrianwong left a comment

Choose a reason for hiding this comment

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

Looks like the new scripts need to be mapped to their Harfbuzz equivalents. Looks fine otherwise.

@wezm wezm force-pushed the wezm/unicode-16 branch from 971ec2a to 0ca6b44 Compare October 4, 2024 02:05
We aren't using it and there are no dependants of this crate listed on
crates.io and not results from a search:
`language:toml yeslogic-unicode-script` on GitHub. It complicates
updates by the dependencies being incompatible with older versions
of rustc.
@wezm wezm force-pushed the wezm/unicode-16 branch from 6b7cd29 to 3ed2a21 Compare October 4, 2024 03:51
@wezm
Copy link
Author

wezm commented Oct 4, 2024

Looks like the new scripts need to be mapped to their Harfbuzz equivalents. Looks fine otherwise.

I dropped the Harfbuzz feature:

We aren't using it and there are no dependants of this crate listed on crates.io and no results from a search: language:toml yeslogic-unicode-script on GitHub. It complicates updates by the dependencies being incompatible with older versions of rustc.

@wezm wezm merged commit 2bdc736 into master Oct 8, 2024
4 checks passed
@wezm wezm deleted the wezm/unicode-16 branch October 8, 2024 01:04
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