2290: Update PullRequest::updateReview

Reviewed-by: erikj
zhaosongzs · Jun 7, 2024 · c40e621 · c40e621
1 parent 3f7de91
commit c40e621
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 5 deletions.
diff --git a/bots/tester/src/test/java/org/openjdk/skara/bots/tester/InMemoryPullRequest.java b/bots/tester/src/test/java/org/openjdk/skara/bots/tester/InMemoryPullRequest.java
@@ -68,7 +68,7 @@ public void addReview(Review.Verdict verdict, String body) {
     }
 
     @Override
-    public void updateReview(int id, String body) {
+    public void updateReview(String id, String body) {
     }
 
     @Override

diff --git a/forge/src/main/java/org/openjdk/skara/forge/PullRequest.java b/forge/src/main/java/org/openjdk/skara/forge/PullRequest.java
@@ -50,7 +50,7 @@ public interface PullRequest extends Issue {
     /**
      * Updates the comment body of a review.
      */
-    void updateReview(int id, String body);
+    void updateReview(String id, String body);
 
     /**
      * Add a file specific comment.

diff --git a/forge/src/main/java/org/openjdk/skara/forge/github/GitHubPullRequest.java b/forge/src/main/java/org/openjdk/skara/forge/github/GitHubPullRequest.java
@@ -181,7 +181,7 @@ public void addReview(Review.Verdict verdict, String body) {
     }
 
     @Override
-    public void updateReview(int id, String body) {
+    public void updateReview(String id, String body) {
         request.put("pulls/" + json.get("number").toString() + "/reviews/" + id)
                .body("body", body)
                .execute();

diff --git a/forge/src/main/java/org/openjdk/skara/forge/gitlab/GitLabMergeRequest.java b/forge/src/main/java/org/openjdk/skara/forge/gitlab/GitLabMergeRequest.java
@@ -199,7 +199,7 @@ public void addReview(Review.Verdict verdict, String body) {
     }
 
     @Override
-    public void updateReview(int id, String body) {
+    public void updateReview(String id, String body) {
         throw new RuntimeException("not implemented yet");
     }
 

diff --git a/test/src/main/java/org/openjdk/skara/test/TestPullRequest.java b/test/src/main/java/org/openjdk/skara/test/TestPullRequest.java
@@ -99,7 +99,7 @@ public void addReview(Review.Verdict verdict, String body) {
     }
 
     @Override
-    public void updateReview(int id, String body) {
+    public void updateReview(String id, String body) {
         throw new RuntimeException("not implemented yet");
     }