Add update_pull_request method (#6)

JuliaWeb · May 18, 2019 · 111e2b0 · 111e2b0
1 parent 3e92f21
commit 111e2b0
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 2 deletions.
diff --git a/src/api.jl b/src/api.jl
@@ -100,6 +100,15 @@ Create a pull request.
 @endpoint create_pull_request(owner::AStr, repo::AStr)
 @endpoint create_pull_request(project::Integer)
 
+"""
+    update_pull_request(::Forge, owner::$AStr, repo::$AStr, number::Integer; kwargs...)
+    update_pull_request(::Forge, project::Integer, number::Integer; kwargs...)
+
+Update a pull request.
+"""
+@endpoint update_pull_request(owner::AStr, repo::AStr, number::Integer)
+@endpoint update_pull_request(project::Integer, number::Integer)
+
 """
     get_commit(::Forge, owner::$AStr, repo::$AStr, ref::$AStr)
     get_commit(::Forge, project::Integer, ref::$AStr)

diff --git a/src/forges/GitHub/pull_requests.jl b/src/forges/GitHub/pull_requests.jl
@@ -128,3 +128,7 @@ into(::GitHubAPI, ::typeof(get_pull_request)) = PullRequest
 endpoint(::GitHubAPI, ::typeof(create_pull_request), owner::AStr, repo::AStr) =
     Endpoint(:POST, "/repos/$owner/$repo/pulls")
 into(::GitHubAPI, ::typeof(create_pull_request)) = PullRequest
+
+endpoint(::GitHubAPI, ::typeof(update_pull_request), owner::AStr, repo::AStr, number::Integer) =
+    Endpoint(:PATCH, "/repos/$owner/$repo/pulls/$number")
+into(::GitHubAPI, ::typeof(update_pull_request)) = PullRequest
diff --git a/src/forges/GitLab/merge_requests.jl b/src/forges/GitLab/merge_requests.jl
@@ -84,10 +84,10 @@ endpoint(::GitLabAPI, ::typeof(get_pull_requests), project::Integer) =
     Endpoint(:GET, "/projects/$project/merge_requests")
 into(::GitLabAPI, ::typeof(get_pull_requests)) = Vector{MergeRequest}
 
-endpoint(::GitLabAPI, ::typeof(get_pull_requests), project::Integer, number::Integer) =
+endpoint(::GitLabAPI, ::typeof(get_pull_request), project::Integer, number::Integer) =
     Endpoint(:GET, "/projects/$project/merge_requests/$number")
 endpoint(
-    ::GitLabAPI, ::typeof(get_pull_requests),
+    ::GitLabAPI, ::typeof(get_pull_request),
     owner::AStr, repo::AStr, number::Integer,
 ) = Endpoint(:GET, "/projects/$(encode(owner, repo))/merge_requests/$number")
 into(::GitLabAPI, ::typeof(get_pull_request)) = MergeRequest
@@ -97,3 +97,11 @@ endpoint(::GitLabAPI, ::typeof(create_pull_request), project::Integer) =
 endpoint(::GitLabAPI, ::typeof(create_pull_request), owner::AStr, repo::AStr) =
     Endpoint(:POST, "/projects/$(encode(owner, repo))/merge_requests")
 into(::GitLabAPI, ::typeof(create_pull_request)) = MergeRequest
+
+endpoint(::GitLabAPI, ::typeof(update_pull_request), project::Integer, number::Integer) =
+    Endpoint(:PUT, "/projects/$project/merge_requests/$number")
+endpoint(
+    ::GitLabAPI, ::typeof(update_pull_request),
+    owner::AStr, repo::AStr, number::Integer,
+) = Endpoint(:PUT, "/projects/$(encode(owner, repo))/merge_requests/$number")
+into(::GitLabAPI, ::typeof(update_pull_request)) = MergeRequest