cyco130/esbuild-plugin-polyfill-node

stream polyfill

tbruun opened this issue · 4 comments

tbruun commented

Hi.
Would you consider replacing the "stream" dependency with "stream-browserify" like so?

The "stream" dependency is very old and uses a very old stream API.

cyco130 commented

I was planning on switching to JSPM polyfills, at least as an option (see #2) but didn't have the time yet.

Would you like to send a PR? Otherwise I can do it in a few days.

tbruun commented

If you're planning on switching I can wait as I have already published a temporary forked package.

With the current version of your dependency any calls to stream.Transform fails (at least for me)

cyco130 commented

OK, let's keep this open. I'll do the switch at the first opportunity or just update stream package is it takes long.

  • 🎉1
cyco130 commented

Hi,

I finally made the switch and released it as version 0.2.0. Hope it solves your problems.

  • ❤️1