Skip to content
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

doc: update missing deprecation #23883

Merged
merged 1 commit into from
Oct 27, 2018
Merged

doc: update missing deprecation #23883

merged 1 commit into from
Oct 27, 2018

Conversation

cjihrig
Copy link
Contributor

@cjihrig cjihrig commented Oct 25, 2018

The deprecation code was not updated when landing the PR.

Refs: #23760

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

@nodejs-github-bot nodejs-github-bot added the net Issues and PRs related to the net subsystem. label Oct 25, 2018
cjihrig referenced this pull request Oct 25, 2018
This is an undocumented utility function that is of questionable
utility.

Fixes: #18391

PR-URL: #23760
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
@jasnell
Copy link
Member

jasnell commented Oct 25, 2018

ugh... forgot about that darn dep code update.

@vsemozhetbyt vsemozhetbyt added deprecations Issues and PRs related to deprecations. author ready PRs that have at least one approval, no pending requests for changes, and a CI started. labels Oct 25, 2018
@vsemozhetbyt
Copy link
Contributor

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

@jasnell
Copy link
Member

jasnell commented Oct 25, 2018

@trivikr trivikr added the fast-track PRs that do not need to wait for 48 hours to land. label Oct 26, 2018
@jasnell
Copy link
Member

jasnell commented Oct 26, 2018

@cjihrig
Copy link
Contributor Author

cjihrig commented Oct 27, 2018

The deprecation code was not updated when landing the PR.

PR-URL: nodejs#23883
Refs: nodejs#23760
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Matheus Marchini <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. deprecations Issues and PRs related to deprecations. fast-track PRs that do not need to wait for 48 hours to land. net Issues and PRs related to the net subsystem.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants