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

feat: use Intl.Segmenter #146

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from
Draft

feat: use Intl.Segmenter #146

wants to merge 8 commits into from

Conversation

mnater
Copy link
Owner

@mnater mnater commented Oct 13, 2020

Notes:
It works but is slow, not well supported and some tests break
[skip travis-ci]

Notes:
It works but is slow, not well supported and some tests break
 [skip travis-ci]
@mnater mnater linked an issue Oct 13, 2020 that may be closed by this pull request
Prohibit call to constructor of Intl.Segmenter in each run of
hyphenateText

[skip travis-ci]
Notes:
It works but is slow, not well supported and some tests break
 [skip travis-ci]
Prohibit call to constructor of Intl.Segmenter in each run of
hyphenateText

[skip travis-ci]
@lgtm-com
Copy link

lgtm-com bot commented Oct 29, 2022

This pull request introduces 1 alert when merging d3efb8f into a418d55 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

Still slover than simple regex-replace (and no support in FF)

[ci skip]
@mnater mnater self-assigned this Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Intl.Segmenter
1 participant