-
Notifications
You must be signed in to change notification settings - Fork 30.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Please document the stdio array in cluster.settings #47705
Labels
cluster
Issues and PRs related to the cluster subsystem.
doc
Issues and PRs related to the documentations.
good first issue
Issues that are suitable for first-time contributors.
Comments
Can you please tell me in more brief explain? |
Would you like to open a PR to address this? Please don't ask for permission or if someone else is working on it, just opening the PR is better. |
aduh95
added
the
good first issue
Issues that are suitable for first-time contributors.
label
May 21, 2023
Karnav123
added a commit
to Karnav123/node
that referenced
this issue
May 22, 2023
Fix issue nodejs#47705
lpinca
pushed a commit
to Karnav123/node
that referenced
this issue
May 29, 2023
targos
pushed a commit
that referenced
this issue
May 30, 2023
Fixes: #47705 PR-URL: #48110 Refs: https://nodejs.org/docs/latest-v20.x/api/cluster.html#clustersettings Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
danielleadams
pushed a commit
that referenced
this issue
Jul 6, 2023
Fixes: #47705 PR-URL: #48110 Refs: https://nodejs.org/docs/latest-v20.x/api/cluster.html#clustersettings Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
MoLow
pushed a commit
to MoLow/node
that referenced
this issue
Jul 6, 2023
Fixes: nodejs#47705 PR-URL: nodejs#48110 Refs: https://nodejs.org/docs/latest-v20.x/api/cluster.html#clustersettings Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
Ceres6
pushed a commit
to Ceres6/node
that referenced
this issue
Aug 14, 2023
Fixes: nodejs#47705 PR-URL: nodejs#48110 Refs: https://nodejs.org/docs/latest-v20.x/api/cluster.html#clustersettings Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
Ceres6
pushed a commit
to Ceres6/node
that referenced
this issue
Aug 14, 2023
Fixes: nodejs#47705 PR-URL: nodejs#48110 Refs: https://nodejs.org/docs/latest-v20.x/api/cluster.html#clustersettings Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
cluster
Issues and PRs related to the cluster subsystem.
doc
Issues and PRs related to the documentations.
good first issue
Issues that are suitable for first-time contributors.
Affected URL(s)
https://nodejs.org/dist/latest-v20.x/docs/api/cluster.html#clustersettings
Description of the problem
The docs state that the
stdio
property ofcluster.settings
is an Array but they do not document what the array should be populated with.It should link to https://nodejs.org/dist/latest-v20.x/docs/api/child_process.html#optionsstdio if it uses the same structure (I’m assuming it does, as
child_process.fork()
does.The text was updated successfully, but these errors were encountered: