destroy/pump

[calvinmetcalf]

since pump relies on destroy, we probably want destroy in before we do pump.

[mcollina]

I would say so, let's wait for nodejs/node#12925 to land.

[mcollina]

Relevant issues:

mafintosh/pump#17
nodejs/node#1834

[calvinmetcalf]

I have a pull I've started a bit back that I can clean up with a bit and submit it, I was waiting on destroy to land first

[mcollina]

@calvinmetcalf I think you can fire that, and we can discuss it in the issue tracker.

[calvinmetcalf]

yeah I need to clean it up, I had been waiting on destroy to land

…

On Tue, Jun 6, 2017 at 9:58 AM Matteo Collina ***@***.***> wrote: @calvinmetcalf <https://github.com/calvinmetcalf> I think you can fire that, and we can discuss it in the issue tracker. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#283 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABE4n1-uE1K8qRsvleCxb2ITQBA_umypks5sBVr6gaJpZM4NT_qE> .

[calvinmetcalf]

nodejs/node#13506

[SimenB]

Any news here? Or is it blocked on landing in node core? Would love to be able to just depend on readable stream 🙂

[mcollina]

@SimenB it's just work that needs doing. Do you want to help a bit?

[SimenB]

My understanding of stream internals are next to none, so I'm unfortunately not much help with that 🙁

[prog1dev]

nodejs/node#12925 landed in Version 8.0.0, 2017-05-30