Closed
Description
https://developer.github.com/v3/pulls/review_requests/
- Need to update the
configure()
method to set theAccept
header - 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
$teamReviewers
parameter and setting the default value of$reviewers
to 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