Handle Streams about gulp-csvtojson HOT 5 CLOSED

The pull request has updated tests, and should properly handle streams now.

Would this be ok to merge in? Do you want to discuss further?

from gulp-csvtojson.

Wonderful work, you almost rewrite the whole code base, I'll look into it ASAP.

BTW: I just observed one unit test failed in your PR.

from gulp-csvtojson.

Please let me know. All unit tests passed on my side. I have also upgraded the packages, so make sure to do an npm install before npm test.

from gulp-csvtojson.

Finally, I changed replace(/\s\n/g,'') to replace(/\s?\n/g,'') to make it work, my test bed is linux which only generate \n.

from gulp-csvtojson.

code merged, so close this issue.

from gulp-csvtojson.