feature KnpLabs#984 Add deprecations to the PR review methods to allow cleanup (acrobat)

This PR was merged into the 3.2.x-dev branch.

Discussion
----------

Fixes KnpLabs#621

This PR adds some deprecations so we can cleanup these classes (unused parameters/methods) in 4.0.

Commits
-------

0f1f10f Add deprecations to the PR review methods to allow cleanup

Author: acrobat

lib/Github/Api/PullRequest/Review.php

@@ -20,6 +20,8 @@ class Review extends AbstractApi
 
     public function configure()
     {
+        trigger_deprecation('KnpLabs/php-github-api', '3.2', 'The "%s" is deprecated and will be removed.', __METHOD__);
+
         return $this;
     }
 
@@ -37,6 +39,10 @@ public function configure()
      */
     public function all($username, $repository, $pullRequest, array $params = [])
     {
+        if (!empty($params)) {
+            trigger_deprecation('KnpLabs/php-github-api', '3.2', 'The "$params" parameter is deprecated, to paginate the results use the "ResultPager" instead.');
+        }
+
         $parameters = array_merge([
             'page' => 1,
             'per_page' => 30,

lib/Github/Api/PullRequest/ReviewRequest.php

@@ -14,6 +14,8 @@ class ReviewRequest extends AbstractApi
 
     public function configure()
     {
+        trigger_deprecation('KnpLabs/php-github-api', '3.2', 'The "%s" is deprecated and will be removed.', __METHOD__);
+
         return $this;
     }
 
@@ -29,6 +31,10 @@ public function configure()
      */
     public function all($username, $repository, $pullRequest, array $params = [])
     {
+        if (!empty($params)) {
+            trigger_deprecation('KnpLabs/php-github-api', '3.2', 'The "$params" parameter is deprecated, to paginate the results use the "ResultPager" instead.');
+        }
+
         return $this->get('/repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/pulls/'.$pullRequest.'/requested_reviewers', $params);
     }