Skip to content

doc: fixup process.binding deprecation code - #22062

Closed
jasnell wants to merge 1 commit into
nodejs:masterfrom
jasnell:fixup-deprecation-code
Closed

doc: fixup process.binding deprecation code#22062
jasnell wants to merge 1 commit into
nodejs:masterfrom
jasnell:fixup-deprecation-code

Conversation

@jasnell

Copy link
Copy Markdown
Member

Fixup deprecation code from #22004

Missed assigning the actual code on landing.

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

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-botnodejs-github-bot added deprecations Issues and PRs related to deprecations. doc Issues and PRs related to the documentations. labels Aug 1, 2018
@jasnelljasnell added the fast-track PRs that do not need to wait for 48 hours to land. label Aug 1, 2018
@targos

Copy link
Copy Markdown
Member

👍 this comment to fast-track

@jasnell

Copy link
Copy Markdown
MemberAuthor

@targos

Copy link
Copy Markdown
Member

@jasnell CI-lite was automatically launched by the bot ;)

@jasnell

Copy link
Copy Markdown
MemberAuthor

heh, those are the same jobs aren't they ;-) ... force of habit starting it manually

@jasnell

Copy link
Copy Markdown
MemberAuthor

getting this landed :-)

jasnell added a commit that referenced this pull request Aug 1, 2018
PR-URL: #22062
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
@jasnell

Copy link
Copy Markdown
MemberAuthor

Landed in dcfbedb

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@jasnell@nodejs-github-bot@targos@apapirovski@ChALkeR@cjihrig@maclover7