Skip to content

Conversation

@rvagg
Copy link
Member

@rvaggrvagg commented Jul 3, 2015

Update AUTHORS list using tools/update-authors.sh

@fastest963 if you'd rather your latest email address be used in the authors list then let me know and I'll switch it.

@bnoordhuis
Copy link
Member

LGTM

@jameshartig
Copy link
Contributor

@rvagg yup if you could update it to [email protected] instead of grooveshark.com that'd be great. Thanks!

@mscdexmscdex added the doc Issues and PRs related to the documentations. label Jul 3, 2015
Update AUTHORS list using tools/update-authors.sh PR-URL: nodejs#2100 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: James Hartig <[email protected]>
@rvaggrvagg merged commit 4614033 into nodejs:masterJul 4, 2015
@rvagg
Copy link
MemberAuthor

rvagg commented Jul 4, 2015

thanks both, 4614033

@rvaggrvagg deleted the update-authors branch July 4, 2015 05:20
@Fishrock123Fishrock123 mentioned this pull request Jul 6, 2015
mscdex pushed a commit to mscdex/io.js that referenced this pull request Jul 9, 2015
Update AUTHORS list using tools/update-authors.sh PR-URL: nodejs#2100 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: James Hartig <[email protected]>
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.

4 participants

@rvagg@bnoordhuis@jameshartig@mscdex