8000 Add configurable support for non-ASCII languages in Slugify by cemilf · Pull Request #163 · ctolkien/Slugify · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add configurable support for non-ASCII languages in Slugify #163

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

Conversation

cemilf
Copy link
Contributor
@cemilf cemilf commented May 17, 2025

Introduced a SupportNonAsciiLanguages option in SlugHelperConfiguration to enable handling of non-ASCII characters. Updated SlugHelper to use this option, replacing the now-obsolete SlugHelperForNonAsciiLanguages class.

Refactored tests and benchmarks to use the new configuration, adding comprehensive test cases for various languages and edge cases. Removed redundant methods and tests tied to the old approach.

Updated project files to reflect version changes, signaling the introduction of this streamlined and unified feature.

Related Work Items: #157

Introduced a `SupportNonAsciiLanguages` option in `SlugHelperConfiguration` to enable handling of non-ASCII characters. Updated `SlugHelper` to use this option, replacing the now-obsolete `SlugHelperForNonAsciiLanguages` class.

Refactored tests and benchmarks to use the new configuration, adding comprehensive test cases for various languages and edge cases. Removed redundant methods and tests tied to the old approach.

Updated project files to reflect version changes, signaling the introduction of this streamlined and unified feature.

Related Work Items: ctolkien#155
@ctolkien ctolkien merged commit 84bcd59 into ctolkien:main May 19, 2025
4 checks passed
@cemilf cemilf deleted the merge-slughelperfornonasciilanguages-into-slughelper branch May 20, 2025 06:20
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
0