FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

doc: add thealphanerd to collaborators by MylesBorins · Pull Request #3723 · nodejs/node · GitHub

/ node Public

doc: add thealphanerd to collaborators - #3723

Closed
MylesBorins wants to merge 1 commit into
nodejs:masterfrom
MylesBorins:add-thealphanerd
Closed

doc: add thealphanerd to collaborators#3723
MylesBorins wants to merge 1 commit into
nodejs:masterfrom
MylesBorins:add-thealphanerd

Conversation

Copy link
Copy Markdown
Contributor

No description provided.

cjihrig commented Nov 9, 2015

Copy link
Copy Markdown
Contributor

LGTM

Copy link
Copy Markdown
Contributor

LGTM. yay

Trott commented Nov 9, 2015

Copy link
Copy Markdown
Member

Trott commented Nov 9, 2015

Copy link
Copy Markdown
Member

Er, uh, actually, aren't the collaborators listed in alphabetical order by GitHub username?

Copy link
Copy Markdown
Contributor

good point

Copy link
Copy Markdown
Contributor Author

fixed

Copy link
Copy Markdown
Contributor Author

Can I get another round of LGTM 😄

Copy link
Copy Markdown
Contributor

LGTM

Fishrock123 added the doc Issues and PRs related to the documentations. label Nov 9, 2015

Trott commented Nov 9, 2015

Copy link
Copy Markdown
Member

LGTM

1 similar comment

saghul commented Nov 9, 2015

Copy link
Copy Markdown
Member

LGTM

Copy link
Copy Markdown
Contributor Author

Landed in 7b89a3d

MylesBorins pushed a commit that referenced this pull request Nov 9, 2015
PR-URL: #3723
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Saúl Ibarra Corretgé <saghul@gmail.com>
MylesBorins closed this Nov 9, 2015
MylesBorins pushed a commit that referenced this pull request Nov 10, 2015
PR-URL: #3723
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Saúl Ibarra Corretgé <saghul@gmail.com>
Fishrock123 mentioned this pull request Nov 11, 2015
MylesBorins pushed a commit that referenced this pull request Nov 17, 2015
PR-URL: #3723
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Saúl Ibarra Corretgé <saghul@gmail.com>

Copy link
Copy Markdown
Contributor Author

landed in v4.x-staging in 64e6d06

MylesBorins pushed a commit that referenced this pull request Dec 4, 2015
PR-URL: #3723
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Saúl Ibarra Corretgé <saghul@gmail.com>
jasnell mentioned this pull request Dec 17, 2015
MylesBorins pushed a commit that referenced this pull request Dec 17, 2015
PR-URL: #3723
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Saúl Ibarra Corretgé <saghul@gmail.com>
MylesBorins pushed a commit that referenced this pull request Dec 23, 2015
PR-URL: #3723
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Saúl Ibarra Corretgé <saghul@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

7 participants


Back | FazBrowse Home | New Git URL