Fix for issue #911

[bezrodnov]

This change should address the performance issue #911 where parsing a file with a lot of empty rows takes more time than expected when skipEmptyLines option is set to greedy.

[pokoli]

Hi @bezrodnov,

Thanks for your PR, it looks good to me but I see you updated the minified version and this is something that we just do on the release processs. Could you please revert this change? Thanks

[bezrodnov]

Hi @pokoli,

Sorry, I did not know if I should build the minified version or not and that's why I put it in a different commit (so that I can easily revert it).

I've removed the minified version from the PR, could you please have a look? 🙂

[julienavert]

Hi and thanks to everyone involved for their work 🙂

I would like to know if there is an estimation of when this fix is going to be merged and published in a new version.

[pokoli]

Hi @bezrodnov ,

We have introduced some fixes on our CI on the master branch, before merging this PR i will like to run the test suite with the latest configuration. Could you please merge the master branch on your changes?

@julienavert sorry for the delay, we have some PR that should be reviewed and merged. Once they land on the master land I will publish a new version containing the fixes.

[bezrodnov]

Hi @pokoli,

Thank you for coming back! 🙂
I've updated the PR with the latest changes from master. Could you please run the CI pipeline on it?

[pokoli]

@bezrodnov it's already merged. Thanks for your contribution!

[pokoli]

@julienavert I just published 5.3.2 on npm which contains this fix (among others).
Hope this helps!