Skip to content

doc: update AUTHORS list#21468

Closed
targos wants to merge 1 commit intonodejs:masterfrom
targos:update-authors
Closed

doc: update AUTHORS list#21468
targos wants to merge 1 commit intonodejs:masterfrom
targos:update-authors

Conversation

@targos
Copy link
Member

@targos targos commented Jun 22, 2018

No description provided.

@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Jun 22, 2018

@targos build started: https://ci.nodejs.org/blue/organizations/jenkins/node-test-pull-request-lite-pipeline/detail/node-test-pull-request-lite-pipeline/123/pipeline

@nodejs-github-bot nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Jun 22, 2018
@trivikr trivikr added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jun 23, 2018
@apapirovski
Copy link
Contributor

apapirovski commented Jun 25, 2018

Landed in c7c5212

apapirovski pushed a commit that referenced this pull request Jun 25, 2018
@targos @apapirovski
doc: update AUTHORS list
c7c5212 
PR-URL: #21468
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
@targos targos deleted the update-authors branch June 25, 2018 06:26
targos added a commit that referenced this pull request Jun 25, 2018
@targos
doc: update AUTHORS list
6cca5a8 
PR-URL: #21468
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
@targos targos mentioned this pull request Jul 3, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jasnell jasnell jasnell approved these changes

@lpinca lpinca lpinca approved these changes

@cjihrig cjihrig cjihrig approved these changes

@trivikr trivikr trivikr approved these changes

+1 more reviewer

@vsemozhetbyt vsemozhetbyt vsemozhetbyt approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. 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.

8 participants

@targos @nodejs-github-bot @apapirovski @jasnell @lpinca @cjihrig @vsemozhetbyt @trivikr