doc: add Note of options.stdio into child_process

[koh110]

add Note of options.stdio into child_process.

Ref: #22892 (comment)

Checklist

• make -j4 test (UNIX), or vcbuild test (Windows) passes
• tests and/or benchmarks are included
• documentation is changed or added
• commit message follows commit guidelines

[koh110]

@addaleax@Trott@bzoz That is a continuation #22892. Would you approve it?

[addaleax]

Thank you!

[bzoz]

Passing file descriptors works fine on Windows, its just sockets that are not supported.

[koh110]

I see. I think first sentence is better. I thought that file handle and file descriptors are slightly different.
So, I would change Passing sockets does not supported on Windows..
@addaleax@cjihrig@bzoz

[addaleax]

But in that case it’s probably not right to put it just under this particular type of option? Passing in a net.Socket object, rather than a file descriptor, is invalid on Windows too then, right?

[Trott]

Suggested change

	does not supported on Windows.
	is not supported on Windows.

[koh110]

Windows allow named pipe. I think Passing in a net.Socket object is valid on Windows.
But I feel like that file descriptors might be intelligible phrase more than Passing sockets.
Umm...

It's valid on Windows.
https://github.com/nodejs/node/blob/master/test/common/index.js#L176-L181

require('net').createServer((conn) => {
stdio: ['ignore', conn, 'ignore']
});
}).listen(common.PIPE, () => {});

[koh110]

I have no preference. How about this change? @addaleax@cjihrig

[addaleax]

CI: https://ci.nodejs.org/blue/organizations/jenkins/node-test-pull-request-lite-pipeline/detail/node-test-pull-request-lite-pipeline/2936/pipeline

[danbev]

Landed in e08ac04.