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

Fix regression in slugify. E.g. slugify('HALO') = '-h-a-l-o' #448

Merged
merged 2 commits into from
Sep 5, 2015
Merged

Fix regression in slugify. E.g. slugify('HALO') = '-h-a-l-o' #448

merged 2 commits into from
Sep 5, 2015

Conversation

janraasch
Copy link
Contributor

This regression was introduced with 4124d03.

So underscore.string v3.2.0 and underscore.string v3.2.1 are currently affected.

janraasch referenced this pull request in slara/generator-reveal Sep 5, 2015
bc. of a change in `slugify`:
The commit esamattis/underscore.string@4124d03
causes a slide with name argument »HTML Attributes« to be named `h-t-m-l-attributes.html`

Will issue a pull request in a second. For now we can downgrade to keep the behaviour we want.
@esamattis esamattis merged commit f98c913 into esamattis:master Sep 5, 2015
@esamattis
Copy link
Owner

Released 3.2.2 to npm with this.

@janraasch janraasch deleted the feature/fix-regression-in-slugify branch September 5, 2015 12:43
@janraasch
Copy link
Contributor Author

Great. Thank you.

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