Better naming

Author: Neil Bartley

lib/bitbucket_rest_api/repos.rb

@@ -14,7 +14,7 @@ class Repos < API
                  :Forks       => 'forks',
     :Commits =>'commits',
     :Download=>'download',
-                     :PullRequests    => 'pull_requests'
+                     :PullRequest    => 'pull_request'
 
 
     DEFAULT_REPO_OPTIONS = {
@@ -79,8 +79,8 @@ def download
     end
 
     # Access to Repos::PullRequests API
-    def pull_requests
-      @pull_requests ||= ApiFactory.new 'Repos::PullRequests'
+    def pull_request
+      @pull_request ||= ApiFactory.new 'Repos::PullRequest'
     end
 
     # List branches

lib/bitbucket_rest_api/repos/pull_request.rb

@@ -0,0 +1,42 @@
+# encoding: utf-8
+
+module BitBucket
+  class Repos::PullRequest < API
+
+    # List pull requests
+    #
+    # = Examples
+    #  bitbucket = BitBucket.new
+    #  bitbucket.repos.pull_request.list 'user-name', 'repo-name'
+    #  bitbucket.repos.pull_request.list 'user-name', 'repo-name' { |status| ... }
+    #
+    def list(user_name, repo_name, params={})
+      _update_user_repo_params(user_name, repo_name)
+      _validate_user_repo_params(user, repo) unless user? && repo?
+      normalize! params
+
+      response = get_request("/2.0/repositories/#{user}/#{repo.downcase}/pullrequests", params)
+      return response unless block_given?
+      response.each { |el| yield el }
+    end
+    alias :all :list
+
+    # List pull request participants
+    #
+    # = Examples
+    #  bitbucket = BitBucket.new
+    #  bitbucket.repos.pull_request.list 'user-name', 'repo-name'
+    #  bitbucket.repos.pull_request.list 'user-name', 'repo-name' { |status| ... }
+    #
+    def participants(user_name, repo_name, pull_request_id, params={})
+      _update_user_repo_params(user_name, repo_name)
+      _validate_user_repo_params(user, repo) unless user? && repo?
+      normalize! params
+
+      response = get_request("/1.0/repositories/#{user}/#{repo.downcase}/pullrequests/#{pull_request_id}/participants", params)
+      return response unless block_given?
+      response.each { |el| yield el }
+    end
+
+  end # Repos::Keys
+end # BitBucket