Skip to content

Conversation

@watson
Copy link
Member

@watsonwatson commented Mar 8, 2018

Closes#18943

Checklist

@nodejs-github-botnodejs-github-bot added the doc Issues and PRs related to the documentations. label Mar 8, 2018
Copy link
Member

@mcollinamcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@TrottTrott added the fast-track PRs that do not need to wait for 48 hours to land. label Mar 8, 2018
@BridgeARBridgeAR added the notable-change PRs with changes that should be highlighted in changelogs. label Mar 8, 2018
watson added a commit that referenced this pull request Mar 8, 2018
Closes: #18943 PR-URL: #19234 Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Jon Moss <[email protected]> Reviewed-By: Roman Reiss <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
@watson
Copy link
MemberAuthor

Landed in 3dfce5c

@watsonwatson closed this Mar 8, 2018
@watsonwatson deleted the addme branch March 8, 2018 17:43
@Trott
Copy link
Member

Trott commented Mar 8, 2018

Welcome! 🎉

(No CI? The onboarding doc says to start a CI for PRs like this.)

@Trott
Copy link
Member

Trott commented Mar 8, 2018

Never mind! I found the CI!

CI: https://ci.nodejs.org/job/node-test-pull-request-lite/238/

@watson
Copy link
MemberAuthor

@Trott thanks 😃yeah I ran the lite CI, as it was just a docs update

targos pushed a commit that referenced this pull request Mar 17, 2018
Closes: #18943 PR-URL: #19234 Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Jon Moss <[email protected]> Reviewed-By: Roman Reiss <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
@targostargos mentioned this pull request Mar 18, 2018
MylesBorins pushed a commit that referenced this pull request Mar 20, 2018
Closes: #18943 PR-URL: #19234 Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Jon Moss <[email protected]> Reviewed-By: Roman Reiss <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
MayaLekova pushed a commit to MayaLekova/node that referenced this pull request May 8, 2018
Closes: nodejs#18943 PR-URL: nodejs#19234 Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Jon Moss <[email protected]> Reviewed-By: Roman Reiss <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
MylesBorins pushed a commit that referenced this pull request Aug 17, 2018
Closes: #18943 PR-URL: #19234 Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Jon Moss <[email protected]> Reviewed-By: Roman Reiss <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
@MylesBorinsMylesBorins mentioned this pull request Aug 17, 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.notable-changePRs with changes that should be highlighted in changelogs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

11 participants

@watson@Trott@mcollina@silverwind@jasnell@cjihrig@maclover7@richardlau@devsnek@BridgeAR@nodejs-github-bot