ChildProcess incorrect types

[rdavison]

It appears Process and ChildProcess modules' stdin, stdout, and stderr values are all the opposite type from what they should be.

[banacorn]

Yes, stdout and stderr should be writable while stdin should be readable

[banacorn]

wondering if this repo is still being maintained

[TheSpyder]

Yes, stdout and stderr should be writable while stdin should be readable

isn't that already how it works?

reason-nodejs/src/ChildProcess.re

Lines 81 to 86 in b7b4134

	[@bs.get] [@bs.return nullable]
	external stderr: t => option(Stream.Writable.t(Buffer.t)) = "stderr";
	[@bs.get] [@bs.return nullable]
	external stdin: t => option(Stream.Readable.t(Buffer.t)) = "stdin";
	[@bs.get] [@bs.return nullable]
	external stdout: t => option(Stream.Writable.t(Buffer.t)) = "stdout";

I'm working on a fork that swaps to ReScript syntax and fixes any niggling little issues, but this one seems to have always been correct.
[edit] now released as rescript-nodejs

[TheSpyder]

I see how I was wrong. Fixed in TheSpyder/rescript-nodejs#4.