-
-
Notifications
You must be signed in to change notification settings - Fork 596
Closed
Labels
Description
https://developer.github.com/v3/pulls/review_requests/
- Need to update the
configure()method to set theAcceptheader - Should check on why the
all()method has a$params- Doesn't appear to be anything in the docs that can be set
- Need to figure out if a short-term, backwards-compatible change should be made to
create()andremove()- By adding a
$teamReviewersparameter and setting the default value of$reviewersto an empty array - How to handle different shapes for preview vs not preview?
- By adding a
- Ideally, we would just remove the
['reviewers' => $reviewers]and make it take a more generic$params- To protect from any future changes and to more closely follow the pattern of other endpoints
- Putting the responsibility on the developer to implement the correct information