-
Notifications
You must be signed in to change notification settings - Fork 9
Add support for github inline reviews #98
Comments
Thanks for putting this in here. We're working on figuring out exactly how pullapprove might integrate with the new GitHub reviews, but it's going to take a little bit of time as there's no API yet for that new feature. We'll let you know when we know more! |
Looks like there are now webhook events for reviews, no word on an API yet but we'd be able to start with this at least: https://developer.github.com/v3/activity/events/types/#pullrequestreviewevent |
Good news -- PullApprove now reacts to the GitHub webhook events for reviews. Right now they only tell us two things:
Those will then translate into approve/reject in PullApprove. It does not:
GitHub review integration is enabled by default but can be disabled if need be: group_defaults:
github_reviews:
enabled: false Deeper integration is on the way as soon as GitHub releases an API! |
Forgot to mention that you must be using .pullapprove.yml |
I've followed https://docs.pullapprove.com/migrating-to-v2/ to do so. We're migrating because v2 is required to recognize GitHub review approvals as LGTM. See dropseed/pullapprove-support#98 (comment)
I've followed https://docs.pullapprove.com/migrating-to-v2/ to do so. We're migrating because v2 is required to recognize GitHub review approvals as LGTM. See dropseed/pullapprove-support#98 (comment)
note that a preview API has been announced: https://developer.github.com/changes/2016-12-14-reviews-api/ |
The new PullApprove v3 is completely rebuilt as a GitHub App and is completely integrated with GitHub reviews. More info here: https://www.pullapprove.com |
Github now added it's own review process.
While I didn't have the time to check it out that much I noticed that these reviews are not recognized as approvals/rejections.
They could be uses as an alternative to regex parsing of all comments
The text was updated successfully, but these errors were encountered: