Skip to content

doc: specify types of listener parameter - #20405

Closed
vsemozhetbyt wants to merge 1 commit into
nodejs:masterfrom
vsemozhetbyt:doc-process-specify-listener-param-type
Closed

doc: specify types of listener parameter#20405
vsemozhetbyt wants to merge 1 commit into
nodejs:masterfrom
vsemozhetbyt:doc-process-specify-listener-param-type

Conversation

@vsemozhetbyt

Copy link
Copy Markdown
Contributor
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

We cannot manage with {any} because message cannot be of Symbol or undefined types.

We cannot manage with `{any}` because `message` cannot be
of `Symbol` or `undefined` types.
@vsemozhetbytvsemozhetbyt added the fast-track PRs that do not need to wait for 48 hours to land. label Apr 29, 2018
@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. process Issues and PRs related to the process subsystem. labels Apr 29, 2018
@vsemozhetbyt

Copy link
Copy Markdown
ContributorAuthor

Node.js Collaborators, please, add 👍 here if you approve fast-tracking.

@vsemozhetbyt

Copy link
Copy Markdown
ContributorAuthor

vsemozhetbyt added a commit that referenced this pull request May 2, 2018
We cannot manage with `{any}` because `message` cannot be
of `Symbol` or `undefined` types.
PR-URL: #20405
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@vsemozhetbyt

Copy link
Copy Markdown
ContributorAuthor

Landed in aaca38a
Thanks for the reviews.

@vsemozhetbyt
vsemozhetbyt deleted the doc-process-specify-listener-param-type branch May 2, 2018 02:59
MylesBorins pushed a commit that referenced this pull request May 4, 2018
We cannot manage with `{any}` because `message` cannot be
of `Symbol` or `undefined` types.
PR-URL: #20405
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@MylesBorinsMylesBorins mentioned this pull request May 8, 2018
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docIssues and PRs related to the documentations.fast-trackPRs that do not need to wait for 48 hours to land.processIssues and PRs related to the process subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@vsemozhetbyt@lpinca@cjihrig@nodejs-github-bot