search

BlessCSS / bless

CSS Post-Processor
blesscss.com
MIT License
282 stars 60 forks source link

Added output of file names with new lines #28

Closed olekenneth closed 9 years ago

olekenneth commented 10 years ago

Added output of file names with new lines and null character. So you easily can embed bless into deploy scripts and CI builds.

paulyoung commented 10 years ago

I'd rather not introduce 2 new flags if possible. Would a single --list flag that adds \n\0 to the end of each filename be acceptable?

olekenneth commented 10 years ago

Yes, this looks better. Will you merge it, if I make the changes?

paulyoung commented 10 years ago

Let's review again after that.

olekenneth commented 10 years ago

It doesn't look like I can add both the \n and \0. If I run xargs I get this:

rm: cannot remove `/root/nodejs/bless.js/test-blessed2.css\n': No such file or directory

I really think the print0 is something that the tool should have. but if you won't accept it with this, I can remove it.

olekenneth commented 10 years ago

What do you think now? I'm just printing out \n

paulyoung commented 9 years ago

Thanks @olekenneth. As focus is on v4 moving forward I'm going to close this one (it's been open a long time!)

We should address this in the next version if you feel it's still important.