Skip to content

Conversation

@jasnell
Copy link
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
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
Member

👍 this comment to fast-track

@jasnell
Copy link
MemberAuthor

@targos
Copy link
Member

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

@jasnell
Copy link
MemberAuthor

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

@jasnell
Copy link
MemberAuthor

getting this landed :-)

jasnell added a commit that referenced this pull request Aug 1, 2018
PR-URL: #22062 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: Сковорода Никита Андреевич <[email protected]> Reviewed-By: Jon Moss <[email protected]>
@jasnell
Copy link
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