Remove empty lines

[demisx]

Is it possible to remove empty lines in files using gulp-replace? I've tried this, but it didn't work:

  .pipe(replace(/^\s*$/, '', { skipBinary: true }))

Thanks.

[lazd]

Hi @demisx, this is a regex question, try posting it on Stack Overflow and you're sure to get some answers!

[lazd]

If you do find you're unable to get it to work after trying other strategies, please re-open the issue.

[demisx]

Hi. Thank you for the quick response. I know that /^\s*$/ regex matches an empty line. It just doesn't work with gulp-replace for some reason. I'll try gulp-remove-empty-lines instead.

[lazd]

@demisx Interesting, I will investigate it.

[lazd]

@demisx that matches the whitespace itself, but it doesn't match the newline. Did you try /^\s*\n/ or something like that?

[demisx]

No, I don't believe I've tried this one yet. And it did work. Sorry for the false alarm -- I need to refresh my memory on regex matches. Stupid me. Totally forgot that $ doesn't match \n.

[lazd]

@demisx it happens to the best of us, I'm glad you solved it!

In the mean time, I am adding a test for this because it IS possible there could be weirdness between a buffered and streamed replace involving a newline, and I want to make sure it's working. Thanks!

[demisx]

@lazd You are awesome man! I wish I could give you more than one star.

[lazd]

@demisx so apparently /^\s*\n/gm is the right way to do it, or /^\s*[\r\n]/gm to also catch legacy Mac line endings. You need the m flag to match on a per-line basis, otherwise ^ matches the start of the string.

That said, the results are inconsistent between streams and buffers! This regex does not work when input is buffered, so this is still an issue!

[lazd]

@demisx I pushed some tests to the blanklines branch, maybe you can have a look and see why the buffered test is failing? That's better than a hundred stars!