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

Bug with Unicode #9

Closed
wooorm opened this issue Mar 18, 2017 · 5 comments
Closed

Bug with Unicode #9

wooorm opened this issue Mar 18, 2017 · 5 comments
Labels

Comments

@wooorm
Copy link
Collaborator

wooorm commented Mar 18, 2017

> S = require('github-slugger');
[Function: BananaSlug]
> s = new S
BananaSlug { occurrences: {} }
> s.slug('I ♥ unicode')
'i-♥-unicode'

...but, GitHub slugs that to i--unicode.

Apparently, this used to work fine: github-slugger used to output i--unicode (compare the last Travis build with the one I just triggered).

It’s rather weird that this changed, I’ve got no idea why...

@wooorm wooorm added the bug label Mar 18, 2017
wooorm referenced this issue in syntax-tree/mdast-util-toc Mar 18, 2017
@wooorm
Copy link
Collaborator Author

wooorm commented Mar 18, 2017

This is introduced in [email protected] in mathiasbynens/emoji-regex@37d8faa, but exposed (as the old code was still used in the npm module) in [email protected] (and no, there was no [email protected] release).

Not sure whether is special though...

@Flet
Copy link
Owner

Flet commented Apr 28, 2017

Should we lock down package.json to a specific version of emoji-regex for now?

@wooorm
Copy link
Collaborator Author

wooorm commented Apr 28, 2017

Yeah I think so!

@wooorm
Copy link
Collaborator Author

wooorm commented Apr 28, 2017

Oh, and do you have ideas for a fix? I’m thinking an overwrite for this character?

wooorm added a commit that referenced this issue May 26, 2017
@Flet Flet closed this as completed in #10 May 26, 2017
@Flet
Copy link
Owner

Flet commented May 26, 2017

published as [email protected]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants