Skip to content
New issue

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

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

Already on GitHub? # to your account

doc: add stefanmb to collaborators #6227

Closed
wants to merge 1 commit into from

Conversation

stefanmb
Copy link
Contributor

Checklist
  • documentation is changed or added
  • the commit message follows commit guidelines
Affected core subsystem(s)

doc

Description of change

Adding myself (stefanmb)

@stefanmb stefanmb added the meta Issues and PRs related to the general management of the project. label Apr 15, 2016
@stefanmb stefanmb self-assigned this Apr 15, 2016
@Trott
Copy link
Member

Trott commented Apr 15, 2016

LGTM!

:shipit:

Welcome!

@@ -199,6 +199,7 @@ information about the governance of the Node.js project, see
* [seishun](https://github.com/seishun) - **Nikolai Vavilov** <vvnicholas@gmail.com>
* [silverwind](https://github.com/silverwind) - **Roman Reiss** <me@silverwind.io>
* [srl295](https://github.com/srl295) - **Steven R Loomis** <srloomis@us.ibm.com>
* [stefanmb](https://github.com/stefanmb) - **Stefan Budeanu** <stefan@budeanu.com>
Copy link
Contributor

@MylesBorins MylesBorins Apr 15, 2016

Choose a reason for hiding this comment

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

nit: wouldn't you be about Steven alphabetically?

edit: don't mind me, nothing to see here

@MylesBorins
Copy link
Contributor

MylesBorins commented Apr 15, 2016

LGTM

@stefanmb
Copy link
Contributor Author

@thealphanerd srl295 goes before stefanmb since r is before t?

@evanlucas
Copy link
Contributor

we have sorted by username in the past.

LGTM

@MylesBorins
Copy link
Contributor

face palm... I forgot it was by username not real name... just ignore me 😄

@stefanmb
Copy link
Contributor Author

@thealphanerd No worries :) Thanks for looking it over!

PR-URL: nodejs#6227
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
@santigimeno
Copy link
Member

LGTM

@stefanmb
Copy link
Contributor Author

stefanmb commented Apr 15, 2016

Landed as 8c48a26

@stefanmb stefanmb closed this Apr 15, 2016
Trott pushed a commit that referenced this pull request Apr 15, 2016
PR-URL: #6227
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
MylesBorins pushed a commit that referenced this pull request Apr 19, 2016
PR-URL: #6227
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
MylesBorins pushed a commit that referenced this pull request Apr 20, 2016
PR-URL: #6227
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
MylesBorins pushed a commit that referenced this pull request Apr 20, 2016
PR-URL: #6227
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
@MylesBorins MylesBorins mentioned this pull request Apr 20, 2016
MylesBorins pushed a commit that referenced this pull request Apr 20, 2016
PR-URL: #6227
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
MylesBorins pushed a commit that referenced this pull request Apr 21, 2016
PR-URL: #6227
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
MylesBorins pushed a commit that referenced this pull request Apr 21, 2016
PR-URL: #6227
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
joelostrowski pushed a commit to joelostrowski/node that referenced this pull request Apr 25, 2016
PR-URL: nodejs#6227
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
jasnell pushed a commit that referenced this pull request Apr 26, 2016
PR-URL: #6227
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
MylesBorins pushed a commit that referenced this pull request Jun 1, 2016
PR-URL: #6227
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
@MylesBorins MylesBorins mentioned this pull request Jun 24, 2016
MylesBorins pushed a commit that referenced this pull request Jun 24, 2016
PR-URL: #6227
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
MylesBorins pushed a commit that referenced this pull request Jun 24, 2016
PR-URL: #6227
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
meta Issues and PRs related to the general management of the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants