Skip to content

Conversation

@refack
Copy link
Contributor

Ref: #12789
Ref: #12790

Checklist
Affected core subsystem(s)

doc

@nodejs-github-botnodejs-github-bot added the doc Issues and PRs related to the documentations. label Jun 7, 2017
@joyeecheung
Copy link
Member

Note that #13284 is coming...(or the other way around)

@refack
Copy link
ContributorAuthor

Note that #13284 is coming...(or the other way around)

I know, it's my small way to congratulate you both 🎉 🎆 🍾

Copy link
Member

@TrottTrott left a comment

Choose a reason for hiding this comment

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

LGTM. I don't think this needs to wait 48 hours.

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

@XadillaX
Copy link
Contributor

Congratulations!

@refack
Copy link
ContributorAuthor

@mcollina@joyeecheung , one of you want to do the honors?

@mcollina
Copy link
Member

Landed as d958bf8 🎉 !

@mcollinamcollina closed this Jun 8, 2017
mcollina pushed a commit that referenced this pull request Jun 8, 2017
Ref: #12789 Ref: #12790 PR-URL: #13534 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Evan Lucas <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Jeremiah Senkpiel <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
@refackrefack deleted the ctc-patch-1 branch June 8, 2017 17:01
addaleax pushed a commit that referenced this pull request Jun 10, 2017
Ref: #12789 Ref: #12790 PR-URL: #13534 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Evan Lucas <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Jeremiah Senkpiel <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
@addaleaxaddaleax mentioned this pull request Jun 10, 2017
@gibfahngibfahn mentioned this pull request Jun 15, 2017
3 tasks
MylesBorins pushed a commit that referenced this pull request Jul 17, 2017
Ref: #12789 Ref: #12790 PR-URL: #13534 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Evan Lucas <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Jeremiah Senkpiel <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
@MylesBorinsMylesBorins mentioned this pull request Jul 18, 2017
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants

@refack@joyeecheung@XadillaX@mcollina@evanlucas@Trott@Fishrock123@tniessen@MylesBorins@nodejs-github-bot