Option to avoid removing repeated replacement

[Alexander Akait]

Usage:

const filenamofy = require('filenamify');

const safeFilename = filenamify('/path/to/file---name.ext', { replacement: '-' });

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

[Sindre Sorhus]

What's the use-case? I don't really see any good reason to preserve duplicate replacement characters like that.

[Alexander Akait]

@sindresorhus create css classes based on filesystem path, useful for css-loader, we already have same logic inside

[Sindre Sorhus]

Alright. PR welcome.

Some name ideas:

• condenseReplacements
• preserveRepeatedReplacements
• removeRepeatedReplacements

[George Gkasdrogkas]

@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.

[George Gkasdrogkas]

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 tream-repeat repo.

[Alessio Orsini]

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.

[Alessio Orsini]

This was fixed in 7d4846f.