Sindre Sorhus
Sindre Sorhus
I like it. I could see myself using this in some places. I would name it `duplexStream()` instead of `duplex()`. I think that's clearer.
Maybe use a `stream.Passthrough`. I haven't really looked into this issue closely though.
Any thoughts on the name? - `.child` - `.process` - `.subprocess` - `.childProcess`
Do nothing. `typeVersions` is meant to add fallbacks.
Not in a way that warns the user I think. We could note it in the readme, but we both know that will not help much.
> Should we add a comment on top of the index.d.ts? Many users experiencing issues with tsc will probably look up that file. Sure > I am also going to...
I like it. I always dread when I have to do anything with Node streams. Observables seem perfect for doing this kind of thing.
It's not possible to intercept the stdout/stderr when you're using `inherit` (Really wish it was though). We could support it on `pipe`, but not sure it belongs here.
Relevant Node.js issue: https://github.com/nodejs/node/issues/8033