octokit 4.6.0 → 4.21.0

data/lib/octokit/client/repositories.rb

@@ -10,7 +10,7 @@ module Octokit
       #
       # @see https://developer.github.com/v3/repos/#get
       # @param repo [Integer, String, Hash, Repository] A GitHub repository
-      # @return [Sawyer::Resource] if a repository exists, false otherwise
+      # @return [Boolean]
       def repository?(repo, options = {})
         !!repository(repo, options)
       rescue Octokit::InvalidRepository
@@ -32,7 +32,7 @@ module Octokit
 
       # Edit a repository
       #
-      # @see https://developer.github.com/v3/repos/#edit
+      # @see https://developer.github.com/v3/repos/#update-a-repository
       # @param repo [String, Hash, Repository] A GitHub repository
       # @param options [Hash] Repository information to update
       # @option options [String] :name Name of the repo
@@ -41,11 +41,15 @@ module Octokit
       # @option options [String] :private `true` makes the repository private, and `false` makes it public.
       # @option options [String] :has_issues `true` enables issues for this repo, `false` disables issues.
       # @option options [String] :has_wiki `true` enables wiki for this repo, `false` disables wiki.
+      # @option options [Boolean] :is_template `true` makes the repository a template, `false` makes it not a template.
       # @option options [String] :has_downloads `true` enables downloads for this repo, `false` disables downloads.
       # @option options [String] :default_branch Update the default branch for this repository.
       # @return [Sawyer::Resource] Repository information
       def edit_repository(repo, options = {})
         repo = Repository.new(repo)
+        if options.include? :is_template
+          options = ensure_api_media_type(:template_repositories, options)
+        end
         options[:name] ||= repo.name
         patch "repos/#{repo}", options
       end
@@ -144,6 +148,7 @@ module Octokit
       # @option options [String] :private `true` makes the repository private, and `false` makes it public.
       # @option options [String] :has_issues `true` enables issues for this repo, `false` disables issues.
       # @option options [String] :has_wiki `true` enables wiki for this repo, `false` disables wiki.
+      # @option options [Boolean] :is_template `true` makes this repo available as a template repository, `false` to prevent it.
       # @option options [String] :has_downloads `true` enables downloads for this repo, `false` disables downloads.
       # @option options [String] :organization Short name for the org under which to create the repo.
       # @option options [Integer] :team_id The id of the team that will be granted access to this repository. This is only valid when creating a repo in an organization.
@@ -155,6 +160,9 @@ module Octokit
         opts = options.dup
         organization = opts.delete :organization
         opts.merge! :name => name
+        if opts.include? :is_template
+          opts = ensure_api_media_type(:template_repositories, opts)
+        end
 
         if organization.nil?
           post 'user/repos', opts
@@ -177,6 +185,37 @@ module Octokit
       end
       alias :delete_repo :delete_repository
 
+      # Transfer repository
+      #
+      # Transfer a repository owned by your organization
+      #
+      # @see https://developer.github.com/v3/repos/#transfer-a-repository
+      # @param repo [Integer, String, Hash, Repository] A GitHub repository
+      # @param new_owner [String] The username or organization name the repository will be transferred to.
+      # @param options [Array<Integer>] :team_ids ID of the team or teams to add to the repository. Teams can only be added to organization-owned repositories.
+      # @return [Sawyer::Resource] Repository info for the transferred repository
+      def transfer_repository(repo, new_owner, options = {})
+        options = ensure_api_media_type(:transfer_repository, options)
+        post "#{Repository.path repo}/transfer", options.merge({ new_owner: new_owner })
+      end
+      alias :transfer_repo :transfer_repository
+
+      # Create a repository for a user or organization generated from a template repository
+      #
+      # @param repo [Integer, String, Hash, Repository] A GitHub template repository
+      # @param name [String] Name of the new repo
+      # @option options [String] :owner Organization or user who the new repository will belong to.
+      # @option options [String] :description Description of the repo
+      # @option options [String] :private `true` makes the repository private, and `false` makes it public.
+      # @option options [Boolean] :include_all_branches `true` copies all branches from the template repository, `false` (default) makes it only copy the master branch.
+      # @return [Sawyer::Resource] Repository info for the new repository
+      def create_repository_from_template(repo, name, options = {})
+        options.merge! :name => name
+        options = ensure_api_media_type(:template_repositories, options)
+        post "#{Repository.path repo}/generate", options
+      end
+      alias :create_repo_from_template :create_repository_from_template
+
       # Hide a public repository
       #
       # @param repo [Integer, String, Hash, Repository] A GitHub repository
@@ -277,6 +316,9 @@ module Octokit
       # Requires authenticated client for private repos.
       #
       # @param repo [Integer, String, Hash, Repository] A GitHub repository.
+      # @option options [String] :affiliation Filters the return array by affiliation.
+      #   Can be one of: <tt>outside</tt>, <tt>direct</tt>, or <tt>all</tt>.
+      #   If not specified, defaults to <tt>all</tt>
       # @return [Array<Sawyer::Resource>] Array of hashes representing collaborating users.
       # @see https://developer.github.com/v3/repos/collaborators/#list-collaborators
       # @example
@@ -346,6 +388,18 @@ module Octokit
         boolean_from_response :get, "#{Repository.path repo}/collaborators/#{collaborator}", options
       end
 
+      # Get a user's permission level for a repo.
+      #
+      # Requires authenticated client
+      #
+      # @return [Sawyer::Resource] Hash representing the user's permission level for the given repository
+      # @see https://developer.github.com/v3/repos/collaborators/#review-a-users-permission-level
+      # @example
+      #   @client.permission_level('octokit/octokit.rb', 'lizzhale')
+      def permission_level(repo, collaborator, options={})
+        get "#{Repository.path repo}/collaborators/#{collaborator}/permission", options
+      end
+
       # List teams for a repo
       #
       # Requires authenticated client that is an owner or collaborator of the repo.
@@ -365,6 +419,39 @@ module Octokit
       alias :repo_teams :repository_teams
       alias :teams :repository_teams
 
+      # List all topics for a repository
+      #
+      # Requires authenticated client for private repos.
+      #
+      # @param repo [Integer, String, Hash, Repository] A GitHub repository.
+      # @return [Sawyer::Resource] representing the topics for given repo
+      # @see https://developer.github.com/v3/repos/#list-all-topics-for-a-repository
+      # @example List topics for octokit/octokit.rb
+      #   Octokit.topics('octokit/octokit.rb')
+      # @example List topics for octokit/octokit.rb
+      #   client.topics('octokit/octokit.rb')
+      def topics(repo, options = {})
+        opts = ensure_api_media_type(:topics, options)
+        paginate "#{Repository.path repo}/topics", opts
+      end
+
+      # Replace all topics for a repository
+      #
+      # Requires authenticated client.
+      #
+      # @param repo [Integer, String, Repository, Hash] A Github repository
+      # @param names [Array] An array of topics to add to the repository.
+      # @return [Sawyer::Resource] representing the replaced topics for given repo
+      # @see https://developer.github.com/v3/repos/#replace-all-topics-for-a-repository
+      # @example Replace topics for octokit/octokit.rb
+      #   client.replace_all_topics('octokit/octokit.rb', ['octocat', 'atom', 'electron', 'API'])
+      # @example Clear all topics for octokit/octokit.rb
+      #   client.replace_all_topics('octokit/octokit.rb', [])
+      def replace_all_topics(repo, names, options = {})
+        opts = ensure_api_media_type(:topics, options)
+        put "#{Repository.path repo}/topics", opts.merge(:names => names)
+      end
+
       # List contributors to a repo
       #
       # Requires authenticated client for private repos.
@@ -499,14 +586,14 @@ module Octokit
       #
       # @param repo [Integer, String, Hash, Repository] A GitHub repository.
       # @param branch [String] Branch name
-      # @option options [Hash] :required_status_checks If not null, the following keys are required:  
-      #   <tt>:include_admins [boolean] Enforce required status checks for repository administrators.</tt>  
-      #   <tt>:strict [boolean] Require branches to be up to date before merging.</tt>  
-      #   <tt>:contexts [Array] The list of status checks to require in order to merge into this branch</tt>  
+      # @option options [Hash] :required_status_checks If not null, the following keys are required:
+      #   <tt>:enforce_admins [boolean] Enforce required status checks for repository administrators.</tt>
+      #   <tt>:strict [boolean] Require branches to be up to date before merging.</tt>
+      #   <tt>:contexts [Array] The list of status checks to require in order to merge into this branch</tt>
       #
       # @option options [Hash] :restrictions If not null, the following keys are required:
-      #   <tt>:users [Array] The list of user logins with push access</tt>  
-      #   <tt>:teams [Array] The list of team slugs with push access</tt>.  
+      #   <tt>:users [Array] The list of user logins with push access</tt>
+      #   <tt>:teams [Array] The list of team slugs with push access</tt>.
       #
       #   Teams and users restrictions are only available for organization-owned repositories.
       # @return [Sawyer::Resource] The protected branch
@@ -553,6 +640,24 @@ module Octokit
         boolean_from_response :delete, "#{Repository.path repo}/branches/#{branch}/protection", opts
       end
 
+      # Rename a single branch from a repository
+      #
+      # Requires authenticated client
+      #
+      # @param repo [Integer, String, Hash, Repository] A GitHub repository.
+      # @param branch [String] Current branch name
+      # @param new_name [String] New branch name
+      # @return [Sawyer::Resource] The renamed branch
+      # @see https://developer.github.com/v3/repos/#rename-a-branch
+      # @example
+      #   @client.rename_branch('octokit/octokit.rb', 'master', 'main')
+      def rename_branch(repo, branch, new_name, options = {})
+        params = {
+          new_name: new_name,
+        }
+        post "#{Repository.path repo}/branches/#{branch}/rename", params.merge(options)
+      end
+
       # List users available for assigning to issues.
       #
       # Requires authenticated client for private repos.
@@ -633,6 +738,62 @@ module Octokit
       def delete_subscription(repo, options = {})
         boolean_from_response :delete, "#{Repository.path repo}/subscription", options
       end
+
+      # Create a repository dispatch event
+      #
+      # @param repo [Integer, String, Hash, Repository] A GitHub repository.
+      # @param event_type [String] A custom webhook event name.
+      # @option options [Hash] :client_payload payload with extra information
+      #   about the webhook event that your action or worklow may use.
+      #
+      # @return [Boolean] True if event was dispatched, false otherwise.
+      # @see https://developer.github.com/v3/repos/#create-a-repository-dispatch-event
+      def dispatch_event(repo, event_type, options = {})
+        boolean_from_response :post, "#{Repository.path repo}/dispatches", options.merge({ event_type: event_type })
+      end
+
+      # Check to see if vulnerability alerts are enabled for a repository
+      #
+      # The authenticated user must have admin access to the repository.
+      #
+      # @param repo [Integer, String, Hash, Repository] A GitHub repository.
+      # @return [Boolean] True if vulnerability alerts are enabled, false otherwise.
+      # @see https://docs.github.com/en/rest/reference/repos#check-if-vulnerability-alerts-are-enabled-for-a-repository
+      #
+      # @example
+      #   @client.vulnerability_alerts_enabled?("octokit/octokit.rb")
+      def vulnerability_alerts_enabled?(repo, options = {})
+        opts = ensure_api_media_type(:vulnerability_alerts, options)
+        boolean_from_response(:get, "#{Repository.path repo}/vulnerability-alerts", opts)
+      end
+
+      # Enable vulnerability alerts for a repository
+      #
+      # @param repo [Integer, String, Hash, Repository] A GitHub repository.
+      # @param options [Hash]
+      #
+      # @return [Boolean] True if vulnerability alerts enabled, false otherwise.
+      # @see https://docs.github.com/en/rest/reference/repos#enable-vulnerability-alerts
+      # @example Enable vulnerability alerts for a repository
+      #   @client.enable_vulnerability_alerts("octokit/octokit.rb")
+      def enable_vulnerability_alerts(repo, options = {})
+        opts = ensure_api_media_type(:vulnerability_alerts, options)
+        boolean_from_response(:put, "#{Repository.path repo}/vulnerability-alerts", opts)
+      end
+
+      # Disable vulnerability alerts for a repository
+      #
+      # @param repo [Integer, String, Hash, Repository] A GitHub repository.
+      # @param options [Hash]
+      #
+      # @return [Boolean] True if vulnerability alerts disabled, false otherwise.
+      # @see https://docs.github.com/en/rest/reference/repos#disable-vulnerability-alerts
+      # @example Disable vulnerability alerts for a repository
+      #   @client.disable_vulnerability_alerts("octokit/octokit.rb")
+      def disable_vulnerability_alerts(repo, options = {})
+        opts = ensure_api_media_type(:vulnerability_alerts, options)
+        boolean_from_response(:delete, "#{Repository.path repo}/vulnerability-alerts", opts)
+      end
     end
   end
 end

data/lib/octokit/client/repository_invitations.rb

@@ -13,9 +13,8 @@ module Octokit
       # @param repo [Integer, String, Hash, Repository] A GitHub repository
       # @param user [String] User GitHub username to add
       # @return [Sawyer::Resource] The repository invitation
-      # @see https://developer.github.com/v3/repos/invitations/#invite-a-user-to-a-repository
+      # @see https://developer.github.com/v3/repos/collaborators/#add-user-as-a-collaborator
       def invite_user_to_repository(repo, user, options = {})
-        options = ensure_api_media_type(:repository_invitations, options)
         put "#{Repository.path repo}/collaborators/#{user}", options
       end
       alias invite_user_to_repo invite_user_to_repository
@@ -28,7 +27,6 @@ module Octokit
       # @return [Array<Sawyer::Resource>] A list of invitations
       # @see https://developer.github.com/v3/repos/invitations/#list-invitations-for-a-repository
       def repository_invitations(repo, options = {})
-        options = ensure_api_media_type(:repository_invitations, options)
         paginate "#{Repository.path repo}/invitations", options
       end
       alias repo_invitations repository_invitations
@@ -42,7 +40,6 @@ module Octokit
       # @return [Boolean] True if the invitation was successfully deleted
       # @see https://developer.github.com/v3/repos/invitations/#delete-a-repository-invitation
       def delete_repository_invitation(repo, invitation_id, options = {})
-        options = ensure_api_media_type(:repository_invitations, options)
         boolean_from_response :delete, "#{Repository.path repo}/invitations/#{invitation_id}", options
       end
       alias delete_repo_invitation delete_repository_invitation
@@ -56,7 +53,6 @@ module Octokit
       # @return [Sawyer::Resource] The updated repository invitation
       # @see https://developer.github.com/v3/repos/invitations/#update-a-repository-invitation
       def update_repository_invitation(repo, invitation_id, options = {})
-        options = ensure_api_media_type(:repository_invitations, options)
         patch "#{Repository.path repo}/invitations/#{invitation_id}", options
       end
       alias update_repo_invitation update_repository_invitation
@@ -68,7 +64,6 @@ module Octokit
       # @return [Array<Sawyer::Resource>] The users repository invitations
       # @see https://developer.github.com/v3/repos/invitations/#list-a-users-repository-invitations
       def user_repository_invitations(options = {})
-        options = ensure_api_media_type(:repository_invitations, options)
         paginate "/user/repository_invitations", options
       end
       alias user_repo_invitations user_repository_invitations
@@ -81,7 +76,6 @@ module Octokit
       # @return [Boolean] True if the acceptance of the invitation was successful
       # @see https://developer.github.com/v3/repos/invitations/#accept-a-repository-invitation
       def accept_repository_invitation(invitation_id, options = {})
-        options = ensure_api_media_type(:repository_invitations, options)
         patch "/user/repository_invitations/#{invitation_id}", options
       end
       alias accept_repo_invitation accept_repository_invitation
@@ -94,7 +88,6 @@ module Octokit
       # @return [Boolean] True if the acceptance of the invitation was successful
       # @see https://developer.github.com/v3/repos/invitations/#decline-a-repository-invitation
       def decline_repository_invitation(invitation_id, options = {})
-        options = ensure_api_media_type(:repository_invitations, options)
         boolean_from_response :delete, "/user/repository_invitations/#{invitation_id}", options
       end
       alias decline_invitation decline_repository_invitation

data/lib/octokit/client/reviews.rb

@@ -0,0 +1,227 @@
+module Octokit
+  class Client
+
+    # Methods for the Reviews API
+    #
+    # @see https://developer.github.com/v3/pulls/reviews/
+    module Reviews
+
+      # List reviews on a pull request
+      #
+      # @param repo [Integer, String, Hash, Repository] A GitHub repository
+      # @param number [Integer] Number ID of the pull request
+      # @see https://developer.github.com/v3/pulls/reviews/#list-reviews-on-a-pull-request
+      #
+      # @example
+      #   @client.pull_request_reviews('octokit/octokit.rb', 2)
+      #
+      # @return [Array<Sawyer::Resource>] Array of Hashes representing the reviews
+      def pull_request_reviews(repo, number, options = {})
+        paginate "#{Repository.path repo}/pulls/#{number}/reviews", options
+      end
+
+      # Get a single review
+      #
+      # @param repo [Integer, String, Hash, Repository] A GitHub repository
+      # @param number [Integer] Number ID of the pull request
+      # @param review [Integer] The id of the review
+      # @see https://developer.github.com/v3/pulls/reviews/#get-a-single-review
+      #
+      # @example
+      #   @client.pull_request_review('octokit/octokit.rb', 825, 6505518)
+      #
+      # @return [Sawyer::Resource] Hash representing the review
+      def pull_request_review(repo, number, review, options = {})
+        get "#{Repository.path repo}/pulls/#{number}/reviews/#{review}", options
+      end
+
+      # Delete a pending review
+      #
+      # @param repo [Integer, String, Hash, Repository] A GitHub repository
+      # @param number [Integer] Number ID of the pull request
+      # @param review [Integer] The id of the review
+      # @see https://developer.github.com/v3/pulls/reviews/#delete-a-pending-review
+      #
+      # @example
+      #   @client.delete_pull_request_review('octokit/octokit.rb', 825, 6505518)
+      #
+      # @return [Sawyer::Resource] Hash representing the deleted review
+      def delete_pull_request_review(repo, number, review, options = {})
+        delete "#{Repository.path repo}/pulls/#{number}/reviews/#{review}", options
+      end
+
+      # Get comments for a single review
+      #
+      # @param repo [Integer, String, Hash, Repository] A GitHub repository
+      # @param number [Integer] Number ID of the pull request
+      # @param review [Integer] The id of the review
+      # @see https://developer.github.com/v3/pulls/reviews/#get-comments-for-a-single-review
+      #
+      # @example
+      #   @client.pull_request_review_comments('octokit/octokit.rb', 825, 6505518)
+      #
+      # @return [Array<Sawyer::Resource>] Array of Hashes representing the review comments
+      def pull_request_review_comments(repo, number, review, options = {})
+        paginate "#{Repository.path repo}/pulls/#{number}/reviews/#{review}/comments", options
+      end
+
+      # Create a pull request review
+      #
+      # @param repo [Integer, String, Hash, Repository] A GitHub repository
+      # @param number [Integer] Number ID of the pull request
+      # @param options [Hash] Method options
+      # @option options [String] :event The review action (event) to perform;
+      #   can be one of APPROVE, REQUEST_CHANGES, or COMMENT.
+      #   If left blank, the review is left PENDING.
+      # @option options [String] :body The body text of the pull request review
+      # @option options [Array<Hash>] :comments Comments part of the review
+      # @option comments [String] :path The path to the file being commented on
+      # @option comments [Integer] :position The position in the file to be commented on
+      # @option comments [String] :body Body of the comment
+      # @see https://developer.github.com/v3/pulls/reviews/#create-a-pull-request-review
+      #
+      # @example
+      #   comments = [
+      #     { path: '.travis.yml', position: 10, body: 'ruby-head is under development that is not stable.' },
+      #     { path: '.travis.yml', position: 32, body: 'ruby-head is also required in thervm section.' },
+      #   ]
+      #   options = { event: 'REQUEST_CHANGES', comments: comments }
+      #   @client.create_pull_request_review('octokit/octokit.rb', 844, options)
+      #
+      # @return [Sawyer::Resource>] Hash respresenting the review
+      def create_pull_request_review(repo, number, options = {})
+        post "#{Repository.path repo}/pulls/#{number}/reviews", options
+      end
+
+      # Submit a pull request review
+      #
+      # @param repo [Integer, String, Hash, Repository] A GitHub repository
+      # @param number [Integer] Number ID of the pull request
+      # @param review [Integer] The id of the review
+      # @param event [String] The review action (event) to perform; can be one of
+      #                       APPROVE, REQUEST_CHANGES, or COMMENT.
+      # @param options [Hash] Method options
+      # @option options [String] :body The body text of the pull request review
+      # @see https://developer.github.com/v3/pulls/reviews/#submit-a-pull-request-review
+      #
+      # @example
+      #   @client.submit_pull_request_review('octokit/octokit.rb', 825, 6505518,
+      #                                      'APPROVE', body: 'LGTM!')
+      #
+      # @return [Sawyer::Resource] Hash respresenting the review
+      def submit_pull_request_review(repo, number, review, event, options = {})
+        options = options.merge(event: event)
+        post "#{Repository.path repo}/pulls/#{number}/reviews/#{review}/events", options
+      end
+
+      # Dismiss a pull request review
+      #
+      # @param repo [Integer, String, Hash, Repository] A GitHub repository
+      # @param number [Integer] Number ID of the pull request
+      # @param review [Integer] The id of the review
+      # @param message [String] The message for the pull request review dismissal
+      # @see https://developer.github.com/v3/pulls/reviews/#dismiss-a-pull-request-review
+      #
+      # @example
+      #   @client.dismiss_pull_request_review('octokit/octokit.rb', 825, 6505518, 'The message.')
+      #
+      # @return [Sawyer::Resource] Hash representing the dismissed review
+      def dismiss_pull_request_review(repo, number, review, message, options = {})
+        options = options.merge(message: message)
+        put "#{Repository.path repo}/pulls/#{number}/reviews/#{review}/dismissals", options
+      end
+
+      # List review requests
+      #
+      # @param repo [Integer, String, Hash, Repository] A GitHub repository
+      # @param number [Integer] Number ID of the pull request
+      # @see https://developer.github.com/v3/pulls/review_requests/#list-review-requests
+      #
+      # @example
+      #   @client.pull_request_review_requests('octokit/octokit.rb', 2)
+      #
+      # @return [Array<Sawyer::Resource>] Array of Hashes representing the review requests
+      def pull_request_review_requests(repo, number, options = {})
+        paginate "#{Repository.path repo}/pulls/#{number}/requested_reviewers", options
+      end
+
+      # Create a review request
+      #
+      # @param repo [Integer, String, Hash, Repository] A GitHub repository
+      # @param number [Integer] Number ID of the pull request
+      # @param reviewers [Hash] :reviewers [Array<String>] An array of user logins
+      # @param options [Hash] :team_reviewers [Array<String>] An array of team slugs
+      # @see https://developer.github.com/v3/pulls/review_requests/#create-a-review-request
+      #
+      # @example
+      #   @client.request_pull_request_review('octokit/octokit.rb', 2, reviewers: ['soudy'])
+      #
+      # @return [Sawyer::Resource>] Hash respresenting the pull request
+      def request_pull_request_review(repo, number, reviewers = {}, options = {})
+        # TODO(5.0): remove deprecated behavior
+        if reviewers.is_a?(Array)
+          octokit_warn(
+            "Deprecated: Octokit::Client#request_pull_request_review "\
+            "no longer takes a separate :reviewers argument.\n" \
+            "Please update your call to pass :reviewers and :team_reviewers as part of the options hash."
+          )
+          options = options.merge(reviewers: reviewers)
+        else
+          options = options.merge(reviewers)
+        end
+
+        post "#{Repository.path repo}/pulls/#{number}/requested_reviewers", options
+      end
+
+      # Delete a review request
+      #
+      # @param repo [Integer, String, Hash, Repository] A GitHub repository
+      # @param id [Integer] The id of the pull request
+      # @param reviewers [Hash] :reviewers [Array] An array of user logins
+      # @param options [Hash] :team_reviewers [Array] An array of team slugs
+      #
+      # @see https://developer.github.com/v3/pulls/review_requests/#delete-a-review-request
+      #
+      # @example
+      #   options = {
+      #     "reviewers" => [ "octocat", "hubot", "other_user" ],
+      #     "team_reviewers" => [ "justice-league" ]
+      #   }
+      #   @client.delete_pull_request_review_request('octokit/octokit.rb', 2, options)
+      #
+      # @return [Sawyer::Resource>] Hash representing the pull request
+      def delete_pull_request_review_request(repo, id, reviewers={}, options = {})
+        # TODO(5.0): remove deprecated behavior
+        if !reviewers.empty? && !options.empty?
+          octokit_warn(
+            "Deprecated: Octokit::Client#delete_pull_request_review_request "\
+            "no longer takes a separate :reviewers argument.\n" \
+            "Please update your call to pass :reviewers and :team_reviewers as part of the options hash."
+          )
+        end
+        # For backwards compatibility, this endpoint can be called with a separate reviewers hash.
+        # If not called with a separate hash, then 'reviewers' is, in fact, 'options'.
+        options = options.merge(reviewers)
+        delete "#{Repository.path repo}/pulls/#{id}/requested_reviewers", options
+      end
+
+      # Update a review request comment
+      #
+      # @param repo [Integer, String, Hash, Repository] A GitHub repository
+      # @param number [Integer] Number ID of the pull request
+      # @param review [Integer] The id of the review
+      # @param body [String] body text of the pull request review.
+      # @param options [Hash] Method options
+      # @see https://developer.github.com/v3/pulls/reviews/#update-a-pull-request-review
+      #
+      # @example
+      #   @client.update_pull_request_review('octokit/octokit.rb', 825, 6505518, 'This is close to perfect! Please address the suggested inline change. And add more about this.')
+      #
+      # @return [Sawyer::Resource] Hash representing the review comment
+      def update_pull_request_review(repo, number, review, body, options = {})
+        options[:body] = body
+        put "#{Repository.path repo}/pulls/#{number}/reviews/#{review}", options
+      end
+    end
+  end
+end

data/lib/octokit/client/search.rb

@@ -12,24 +12,39 @@ module Octokit
       # @param options [Hash] Sort and pagination options
       # @option options [String] :sort Sort field
       # @option options [String] :order Sort order (asc or desc)
-      # @option options [Fixnum] :page Page of paginated results
-      # @option options [Fixnum] :per_page Number of items per page
+      # @option options [Integer] :page Page of paginated results
+      # @option options [Integer] :per_page Number of items per page
       # @return [Sawyer::Resource] Search results object
       # @see https://developer.github.com/v3/search/#search-code
       def search_code(query, options = {})
         search "search/code", query, options
       end
 
+      # Search commits
+      #
+      # @param query [String] Search terms and qualifiers
+      # @param options [Hash] Sort and pagination options
+      # @option options [String] :sort Sort field
+      # @option options [String] :order Sort order (asc or desc)
+      # @option options [Integer] :page Page of paginated results
+      # @option options [Integer] :per_page Number of items per page
+      # @return [Sawyer::Resource] Search results object
+      # @see https://developer.github.com/v3/search/#search-commits
+      def search_commits(query, options = {})
+        options = ensure_api_media_type(:commit_search, options)
+        search "search/commits", query, options
+      end
+
       # Search issues
       #
       # @param query [String] Search term and qualifiers
       # @param options [Hash] Sort and pagination options
       # @option options [String] :sort Sort field
       # @option options [String] :order Sort order (asc or desc)
-      # @option options [Fixnum] :page Page of paginated results
-      # @option options [Fixnum] :per_page Number of items per page
+      # @option options [Integer] :page Page of paginated results
+      # @option options [Integer] :per_page Number of items per page
       # @return [Sawyer::Resource] Search results object
-      # @see https://developer.github.com/v3/search/#search-issues
+      # @see https://developer.github.com/v3/search/#search-issues-and-pull-requests
       def search_issues(query, options = {})
         search "search/issues", query, options
       end
@@ -40,8 +55,8 @@ module Octokit
       # @param options [Hash] Sort and pagination options
       # @option options [String] :sort Sort field
       # @option options [String] :order Sort order (asc or desc)
-      # @option options [Fixnum] :page Page of paginated results
-      # @option options [Fixnum] :per_page Number of items per page
+      # @option options [Integer] :page Page of paginated results
+      # @option options [Integer] :per_page Number of items per page
       # @return [Sawyer::Resource] Search results object
       # @see https://developer.github.com/v3/search/#search-repositories
       def search_repositories(query, options = {})
@@ -55,8 +70,8 @@ module Octokit
       # @param options [Hash] Sort and pagination options
       # @option options [String] :sort Sort field
       # @option options [String] :order Sort order (asc or desc)
-      # @option options [Fixnum] :page Page of paginated results
-      # @option options [Fixnum] :per_page Number of items per page
+      # @option options [Integer] :page Page of paginated results
+      # @option options [Integer] :per_page Number of items per page
       # @return [Sawyer::Resource] Search results object
       # @see https://developer.github.com/v3/search/#search-users
       def search_users(query, options = {})

data/lib/octokit/client/source_import.rb

@@ -132,7 +132,7 @@ module Octokit
       #
       # @param repo [Integer, String, Hash, Repository] A GitHub repository.
       # @param options [Hash]
-      # @option options [Fixnum] :page Page of paginated results
+      # @option options [Integer] :page Page of paginated results
       # @return [Array<Sawyer::Resource>] Array of hashes representing files over 100MB.
       # @see https://developer.github.com/v3/migration/source_imports/#get-large-files
       #

data/lib/octokit/client/stats.rb

@@ -88,6 +88,7 @@ module Octokit
       # @return [Array<Sawyer::Resource> or nil] Stats in metric-specific format, or nil if not yet calculated.
       # @see https://developer.github.com/v3/repos/statistics/
       def get_stats(repo, metric, options = {})
+        options = options.dup
         if retry_timeout = options.delete(:retry_timeout)
           retry_wait = options.delete(:retry_wait) || 0.5
           timeout = Time.now + retry_timeout
@@ -95,6 +96,7 @@ module Octokit
         loop do
           data = get("#{Repository.path repo}/stats/#{metric}", options)
           return data if last_response.status == 200
+          return [] if last_response.status == 204
           return nil unless retry_timeout
           return nil if Time.now >= timeout
           sleep retry_wait if retry_wait

data/lib/octokit/client/statuses.rb

@@ -13,7 +13,7 @@ module Octokit
       # @return [Array<Sawyer::Resource>] A list of statuses
       # @see https://developer.github.com/v3/repos/statuses/#list-statuses-for-a-specific-ref
       def statuses(repo, sha, options = {})
-        get "#{Repository.path repo}/statuses/#{sha}", options
+        paginate "#{Repository.path repo}/statuses/#{sha}", options
       end
       alias :list_statuses :statuses
 
@@ -39,7 +39,7 @@ module Octokit
       # @return [Sawyer::Resource] A status
       # @see https://developer.github.com/v3/repos/statuses/#create-a-status
       def create_status(repo, sha, state, options = {})
-        options.merge!(:state => state)
+        options = options.merge(:state => state)
         post "#{Repository.path repo}/statuses/#{sha}", options
       end
     end