doc: link cluster worker.send() to child.send()#839
doc: link cluster worker.send() to child.send()#839sam-github wants to merge 1 commit intonodejs:v1.xfrom
Conversation
4e3951d to
a419c5a
Compare
|
Should help with #774 (comment) |
doc/api/cluster.markdown
Outdated
There was a problem hiding this comment.
i'd say "should use" -> something along the lines of "can be used"
a419c5a to
c5a3859
Compare
|
@brendanashworth PTAL, I incorporated your suggestions. |
|
@brendanashworth, please take a look |
|
@sam-github sorry - I just feel a little bit iffy on OK-ing this immediately because I'd like for these docs to also mention whether it is sync or async, but that would depend on #774. |
There was a problem hiding this comment.
is there anything that we can link process.send() to?
There was a problem hiding this comment.
No. But that's a seperate problem.
|
I don't think the docs should mention if its sync or async, they should say its identical to child.send(), link to child.send(), and leave it at that. Repeating implementation details of node APIs throughout node leads to inevitable inaccuracies. |
There was a problem hiding this comment.
can we link to a code'ified (with brackets)child.send()` rather than plaintext?
There was a problem hiding this comment.
Don't know, but I grepped, and there are no such examples of such markup in existing node docs, this link is in the same form as all other links: git grep '().*\].*html#' doc/api
|
LGTM, I'll merge in a second. |
PR-URL: #839 Reviewed-By: Brendan Ashworth <brendan.ashworth@me.com>
|
Sorry for taking so long on this one. Merged per |
/to @brendanashworth Is this clearer?