Skip to content

Conversation

@oyyd
Copy link
Contributor

@oyydoyyd commented Nov 18, 2018

Refs: #23329

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

@oyyd sadly an error occured when I tried to trigger a build :(

@nodejs-github-botnodejs-github-bot added deprecations Issues and PRs related to deprecations. doc Issues and PRs related to the documentations. labels Nov 18, 2018
@vsemozhetbytvsemozhetbyt added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Nov 18, 2018
@vsemozhetbyt
Copy link
Contributor

@vsemozhetbyt
Copy link
Contributor

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

@oyydoyyd added the fast-track PRs that do not need to wait for 48 hours to land. label Nov 18, 2018
@oyyd
Copy link
ContributorAuthor

oyyd commented Nov 18, 2018

Landed in 34eccb2. Thanks!

@oyydoyyd closed this Nov 18, 2018
pullbot pushed a commit to shakir-abdo/node that referenced this pull request Nov 18, 2018
PR-URL: nodejs#24458 Refs: nodejs#23329 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Vse Mozhet Byt <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
refack pushed a commit to refack/node that referenced this pull request Jan 14, 2019
PR-URL: nodejs#24458 Refs: nodejs#23329 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Vse Mozhet Byt <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.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.

6 participants

@oyyd@nodejs-github-bot@vsemozhetbyt@lpinca@targos@richardlau