browserify-shim icon indicating copy to clipboard operation
browserify-shim copied to clipboard

update to a maintained `through`?

Open ljharb opened this issue 2 years ago • 3 comments

Hi! You depend on through, an excellent package that unfortunately has not been maintained since 2018. I've forked it as https://npmjs.com/@ljharb/through so that I can maintain it (with @dominictarr's blessing), and am using it in https://npmjs.com/tape.

Would you be interested in a PR that switches this project to using my fork instead of the original? It's effectively identical, but it'd ensure any vulnerabilities will be addressed in the future.

ljharb avatar Jul 18 '23 22:07 ljharb

switches inquirer

Assuming this is a copy paste error? But sure this sounds fine happy to merge.

bendrucker avatar Jul 18 '23 22:07 bendrucker

lol oops, yes, copy-paste error. thanks, i'll make a PR!

ljharb avatar Jul 18 '23 22:07 ljharb

Just to clarify: the codebases are (as far as I can tell) effectively identical at this time, except for the fact that through has 0 dependencies whereas @ljharb/through has 12 nested dependencies for the purpose of Node 0.4 support. You can decide for yourself whether the extra dependencies are worth the Node 0.4 support, but I think it is important to be transparent.

valadaptive avatar Jun 23 '24 09:06 valadaptive