Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

wip: replace utf-8 encoder (add unicode-normalization, graphscheme clustering,..) #646

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

wip: replace utf-8 encoder (add unicode-normalization, graphscheme clustering,..) #646

wants to merge 1 commit into from
+73 −133

Conversation

raphamorim
Copy link
Owner

No description provided.

@orhun
Copy link
Contributor

orhun commented Sep 5, 2024

just curious about the rationale about this change 🐻

@raphamorim
Copy link
Owner Author

there's few things rio doesn't do and few bugs due to lack of normalisation, like #191

@raphamorim
Copy link
Owner Author

I am also revisiting performance on this crate

@raphamorim raphamorim force-pushed the main branch 2 times, most recently from 8f38185 to 6f1ea14 Compare February 24, 2025 21:09
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@raphamorim @orhun