search

aceface / crayon

MIT License
10 stars 1 forks source link

has-color and strip-ansi #3

Closed ronkorving closed 9 years ago

ronkorving commented 9 years ago

Hi, I love this module's minimalism (compared to the other terminal color modules out there) and that it doesn't mess with my String.prototype object. But in that spirit, why bundle has-color and strip-ansi? You don't actually use it at all in this project it seems. So if people need it, why not let them install it themselves?

ccheever commented 9 years ago

has-color does get used; if you run this on a terminal that doesn't support color, the module will by default do nothing. strip-ansi could be removed. making a fork of this that has neither of them should be trivial if you want to do that! if i get a chance, i might remove strip-ansi. feel free to submit a patch if you like.