Being able to search for things my review is requested on would be good companion feature for it.
This is one of the features I love about https://reviewable.io.
Introducing review requests
11–16 of 16 posts
Re: Introducing review requests
#12Are there any api endpoints now for querying reviews? I'd love to have a way to get a list of reviews i'm being asked to do.
Re: Introducing review requests
#13Kind of interesting. What I'd really like to be able to do is require specific people to review before merging, for example require a review by at least one of our senior developers. Right now the greenest guy on the team can approve a PR and it can be merged. The real dream would be to be able to set the requirements on a per-directory basis, but I doubt that would ever happen.
FWIW https://reviewable.io supports arbitrarily complex review completion requirements, and the outcome gets fed into a PR status that you can make required as well. Right now the completion condition must be self-contained but I'm considering passing in designated metadata files so you could have, e.g., per-directory "owners" like Google does.
(Disclosure: I'm the founder.)
Re: Introducing review requests
#14Has GitHub improved reviews since launch? I tried the review feature at launch and found it to be a severe regression from plain PR inline comments, because the review comments weren't hidden when the code they referenced changed, so I had to manually check each review comment to see whether or not it was addressed by a follow-on commit. I emailed their support asking if I was using the review feature wrong, and they…
Heh, I never cease to wonder at the variety of -- often contradictory -- code review workflows. For me and many others automatically disappearing comments are a misfeature, since we _want_ to verify that a comment was correctly addressed. To each their own... :)
Re: Introducing review requests
#15Are there any api endpoints now for querying reviews? I'd love to have a way to get a list of reviews i'm being asked to do.
AFAICT the PR review REST API is still on their "near-term roadmap" (https://developer.github.com/early-access/platform-roadmap/).
Re: Introducing review requests
#16Has GitHub improved reviews since launch? I tried the review feature at launch and found it to be a severe regression from plain PR inline comments, because the review comments weren't hidden when the code they referenced changed, so I had to manually check each review comment to see whether or not it was addressed by a follow-on commit. I emailed their support asking if I was using the review feature wrong, and they…
Heh, I never cease to wonder at the variety of -- often contradictory -- code review workflows. For me and many others automatically disappearing comments are a misfeature, since we _want_ to verify that a comment was correctly addressed. To each their own... :)
At the launch of GitHub reviews, it's not just that the outdated comments weren't hidden, but at that time, there were no tools in place of that to collapse threads, mark sections of diff as approved, or otherwise resolve comments with follow-on pushes. That's why I asked their support if I was using it correctly, I was super confused. I'm definitely interested in a better GitHub workflow.