Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc: add gibfahn to collaborators #8533

Closed
wants to merge 1 commit into from

Conversation

gibfahn
Copy link
Member

@gibfahn gibfahn commented Sep 14, 2016

Checklist
  • make lint (UNIX) passes
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

doc

Description of change

Add Gibson Fahnestock to collaborators

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Sep 14, 2016
@Trott
Copy link
Member

Trott commented Sep 14, 2016

LGTM

@gibfahn
Copy link
Member Author

gibfahn commented Sep 14, 2016

@MylesBorins
Copy link
Contributor

wahoo! 🎉

On Wed, Sep 14, 2016, 6:42 PM Rich Trott [email protected] wrote:

LGTM


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#8533 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAecV5Es5zBRaxh7BK76ZSY98VX369ikks5qqCQOgaJpZM4J8_No
.

@lpinca
Copy link
Member

lpinca commented Sep 14, 2016

LGTM

@jbergstroem
Copy link
Member

LGTM! Good to have you on board.

gibfahn added a commit that referenced this pull request Sep 14, 2016
PR-URL: #8533
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
@gibfahn
Copy link
Member Author

gibfahn commented Sep 14, 2016

Landed in 5ce117b

@gibfahn gibfahn closed this Sep 14, 2016
@bnoordhuis
Copy link
Member

Belated LGTM and welcome aboard, Gibson.

@gibfahn gibfahn deleted the pr-add-collaborator branch September 14, 2016 18:40
@mhdawson
Copy link
Member

Same from me too.

MylesBorins pushed a commit that referenced this pull request Sep 30, 2016
PR-URL: #8533
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
MylesBorins pushed a commit that referenced this pull request Oct 10, 2016
PR-URL: #8533
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Fishrock123 pushed a commit that referenced this pull request Oct 11, 2016
PR-URL: #8533
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
rvagg pushed a commit that referenced this pull request Oct 18, 2016
PR-URL: #8533
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
MylesBorins pushed a commit that referenced this pull request Oct 26, 2016
PR-URL: #8533
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
@MylesBorins MylesBorins mentioned this pull request Oct 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants