-
Notifications
You must be signed in to change notification settings - Fork 13
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
Allow displaying Github reviews #5
Comments
I'm all for it. GitHub's API documentation is here. Here's the section on Pull Requests. A quick glance doesn't show a quick and easy way of getting the list of open pull requests for a user across repositories, so we might need to first query for the repository list, and then ask each of them for an open pull request count. It might be possible to work with the Notification API to get that information as well. |
This information is already present, per-user, in https://github.com/pulls/review-requested so surely we can either use the API or scrape that. |
Their v4 API is GraphQL based, which IIUC allows getting this stuff without scraping, even if the REST API doesn't support it. There's even a neat tool to try queries with: https://developer.github.com/v4/explorer/ |
Found a stackoverflow answer with the correct snippet (just need to sub in the user for 'refack'): https://stackoverflow.com/questions/48255251/how-to-query-review-requests-by-user-using-githubs-v4-graphql-api |
Actually, https://developer.github.com/v3/search/#search-issues seems to work for q = 'type:pr review-requested:$username is:open' |
Fetch and display pending github review requests. Fixes #5
I do some of my work in mozilla-central (which means BZ and Phab reviews) but a lot of my work in github.com/mozilla/application-services, for example (which means GH reviews). Can we generalize to a third system?
The text was updated successfully, but these errors were encountered: