GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — doc: update AUTHORS list by BridgeAR · Pull Request #17805 · nodejs/node

Conversation

@BridgeAR

@BridgeAR BridgeAR commented Dec 21, 2017

Copy link
Copy Markdown
Member

Updating the list (+24 authors) and fixing @XadillaX and my entries.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

doc

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
XadillaX reacted with thumbs up emoji
All reactions
  • 👍 1 reaction
@BridgeAR
doc: update AUTHORS list
6e5a3d5
@nodejs-github-bot nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Dec 21, 2017
targos
targos approved these changes Dec 21, 2017
View reviewed changes
cjihrig
cjihrig approved these changes Dec 21, 2017
View reviewed changes
danbev
danbev approved these changes Dec 21, 2017
View reviewed changes
@BridgeAR
BridgeAR requested a review from XadillaX December 21, 2017 13:47
gireeshpunathil
gireeshpunathil approved these changes Dec 21, 2017
View reviewed changes
jasnell
jasnell approved these changes Dec 21, 2017
View reviewed changes
benjamingr
benjamingr reviewed Dec 21, 2017
View reviewed changes
Comment thread AUTHORS
Jure Triglav <[email protected]>
alnyan <[email protected]>
rt33 <[email protected]>
Ulmanb <[email protected]>

@benjamingr benjamingr Dec 21, 2017

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

@Ulmanb

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
Ulmanb and trszdev reacted with hooray emoji
All reactions
  • 🎉 2 reactions

@BridgeAR BridgeAR Dec 21, 2017

Copy link
Copy Markdown
Member Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

?

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
TimothyGu
TimothyGu approved these changes Dec 21, 2017
View reviewed changes
BridgeAR added a commit to BridgeAR/node that referenced this pull request Dec 26, 2017
@BridgeAR
doc: update AUTHORS list
183f072 
PR-URL: nodejs#17805
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
@BridgeAR

BridgeAR commented Dec 26, 2017

Copy link
Copy Markdown
Member Author

Landed in 183f072

(I ran the script again before landing and the following authors were added as well)

+xortiz <[email protected]>
+Waleed Ashraf <[email protected]>
+Mir Mufaqam Ali <[email protected]>
+Nicholas Drane <[email protected]>
+Shobhit Chittora <[email protected]>
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@BridgeAR BridgeAR closed this Dec 26, 2017
@XadillaX

XadillaX commented Dec 28, 2017

Copy link
Copy Markdown
Contributor

Thx!
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
MylesBorins pushed a commit that referenced this pull request Jan 8, 2018
@BridgeAR @MylesBorins
doc: update AUTHORS list
fa0920e 
PR-URL: #17805
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
MylesBorins pushed a commit that referenced this pull request Jan 9, 2018
@BridgeAR @MylesBorins
doc: update AUTHORS list
727ef47 
PR-URL: #17805
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
MylesBorins pushed a commit that referenced this pull request Jan 9, 2018
@BridgeAR @MylesBorins
doc: update AUTHORS list
59711ae 
PR-URL: #17805
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
@MylesBorins MylesBorins mentioned this pull request Jan 10, 2018
v9.4.0 proposal #18069
Merged
gibfahn pushed a commit that referenced this pull request Jan 24, 2018
@BridgeAR @gibfahn
doc: update AUTHORS list
fa530fe 
PR-URL: #17805
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
gibfahn pushed a commit that referenced this pull request Jan 24, 2018
@BridgeAR @gibfahn
doc: update AUTHORS list
4dcb1bb 
PR-URL: #17805
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
@gibfahn gibfahn added land-on-v6.x labels Jan 24, 2018
gibfahn pushed a commit that referenced this pull request Jan 24, 2018
@BridgeAR @gibfahn
doc: update AUTHORS list
f154e76 
PR-URL: #17805
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
@MylesBorins MylesBorins mentioned this pull request Jan 24, 2018
v6.13.0 proposal #18342
Merged
MylesBorins pushed a commit that referenced this pull request Feb 11, 2018
@BridgeAR @MylesBorins
doc: update AUTHORS list
307f051 
PR-URL: #17805
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
MylesBorins pushed a commit that referenced this pull request Feb 12, 2018
@BridgeAR @MylesBorins
doc: update AUTHORS list
c96a871 
PR-URL: #17805
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
MylesBorins pushed a commit that referenced this pull request Feb 13, 2018
@BridgeAR @MylesBorins
doc: update AUTHORS list
f6b4aa6 
PR-URL: #17805
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
@BridgeAR
BridgeAR deleted the update-authors branch April 1, 2019 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@benjamingr benjamingr benjamingr left review comments
@jasnell jasnell jasnell approved these changes
@cjihrig cjihrig cjihrig approved these changes
@gireeshpunathil gireeshpunathil gireeshpunathil approved these changes
@XadillaX XadillaX Awaiting requested review from XadillaX
+3 more reviewers
@danbev danbev danbev approved these changes
@TimothyGu TimothyGu TimothyGu approved these changes
@targos targos targos approved these changes
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

meta Issues and PRs related to the general management of the project.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

11 participants

@BridgeAR @XadillaX @danbev @jasnell @benjamingr @TimothyGu @targos @cjihrig @gireeshpunathil @gibfahn @nodejs-github-bot
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.