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: update AUTHORS list #713

Merged
merged 1 commit into from
Feb 11, 2015
Merged

doc: update AUTHORS list #713

merged 1 commit into from
Feb 11, 2015

Conversation

rvagg
Copy link
Member

@rvagg rvagg commented Feb 4, 2015

Update AUTHORS file using tools/update-authors.sh

I think I've also discovered that my clever Awk program doesn't work properly on OSX, something about ordering I think. Need to fix that somehow. For now I'm just doing an update.

@chrisdickinson
Copy link
Contributor

LGTM

@@ -691,5 +691,14 @@ Xiaowei Li <446240525@qq.com>
toastynerd <tylermorgan86@gmail.com>
Todd Kennedy <toddself@users.noreply.github.com>
Icer Liang <wizicer@users.noreply.github.com>
Stephen Belanger <admin@stephenbelanger.com>
Fishrock123 <fishrock123@rocketmail.com>
Copy link
Member

Choose a reason for hiding this comment

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

Can we add a .mailmap entry for @Fishrock123?

@Fishrock123
Copy link
Member

@rvagg Can we get this in for 1.2.0?

@rvagg
Copy link
Member Author

rvagg commented Feb 10, 2015

updated to current HEAD with 4 more entries and a .mailmap entry for @Fishrock123, will wait to hear from @3674823674252 before I merge this.

Update AUTHORS file using tools/update-authors.sh

PR-URL: nodejs#713
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
@rvagg rvagg merged commit d832be4 into nodejs:v1.x Feb 11, 2015
@rvagg
Copy link
Member Author

rvagg commented Feb 11, 2015

merged @ d832be4, will update .mailmap next time if @3674823674252 expresses an opinion but I'll leave it as is for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants