-
-
Notifications
You must be signed in to change notification settings - Fork 30
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
Option to avoid removing repeated replacement #10
Comments
What's the use-case? I don't really see any good reason to preserve duplicate replacement characters like that. |
@sindresorhus create css classes based on filesystem path, useful for css-loader, we already have same logic inside |
Alright. PR welcome. Some name ideas:
|
@sindresorhus the replacement is done at filenamify.js#L30 using the tream-repeat library. We should implement the change in that library first I believe. I will open an issue and link it with this one. |
It seems that I missunderstood the question. I though we wanted to replace all the occurances of repeated characters, except a specific one. I'm closing the issue on |
Is there any particular reason why #18 has been refused? I have a similar fix for this issue and was wondering if it would be welcomed. |
This was fixed in 7d4846f. |
Usage:
Now:
/path/to/file---name.ext
->path-to-file-name.ext
Feature request (maybe
keepRepeat: true
or better name):/path/to/file---name.ext
->path-to-file---name.ext
The text was updated successfully, but these errors were encountered: