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 #2100

Merged
merged 1 commit into from
Jul 4, 2015
Merged

doc: update AUTHORS list #2100

merged 1 commit into from
Jul 4, 2015

Conversation

rvagg
Copy link
Member

@rvagg rvagg 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!

@mscdex mscdex 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]>
@rvagg rvagg merged commit 4614033 into nodejs:master Jul 4, 2015
@rvagg
Copy link
Member Author

rvagg commented Jul 4, 2015

thanks both, 4614033

@rvagg rvagg deleted the update-authors branch July 4, 2015 05:20
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 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.

4 participants