-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Support for creating reaction for a release #2196
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2196 +/- ##
=======================================
Coverage 97.78% 97.78%
=======================================
Files 112 112
Lines 9976 9992 +16
=======================================
+ Hits 9755 9771 +16
Misses 154 154
Partials 67 67
Continue to review full report at Codecov.
|
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.
Thank you, @viveknathani !
LGTM.
Awaiting second LGTM (from any other go-github contributor) before merging.
@gmlewis It seems like the requested reviewer has not been active in this repository for quite some time (as per the pending PRs). Can we get somebody else to do the review if that is okay? |
Hi @viveknathani . Any contributor to this repo is welcome to review and approve PRs, but there are only a small number of people on the official "Reviewers" list that can be requested and I unfortunately can not modify that list like I could in the past. |
All right @gmlewis. @ganeshkumarsv, @devfd, @sushmitaw, @AGMETEOR could you do the code review over here since you have been a recent contributor to this repository? |
@gmlewis Is there any other way we can move forward from here? Getting a second voluntary reviewer is not working out... |
Sorry for any delays. I approve the changes. |
Thank you, @AGMETEOR ! |
closes #2194