ExtendedCommitComment.java

- now extends CommitComment, yay
- but they are still missing a few things..

README.md
- updated, seems line is now removed from commit comments

Author: aaronjwhiteside

README.md

@@ -335,19 +335,22 @@ updatedAt | `Date` | false
 ### Properties
 Name | Type | Setter | Description
 -----|------|----------|------------
-id | `Integer` | false
+id | `Long` | false
 url | `String` | false
 user | `String` | false
 createdAt | `Date` | false
 updatedAt | `Date` | false
 commitId | `String` | false
-originalCommitId | `Integer` | false
+originalCommitId | `Long` | false
 body | `String` | **true**
 path | `String` | false
-line | `Integer` | false
+line | `Integer` | false | This will always return `null` as the GitHub APIs no longer return `line`, use `position` instead.
 position | `Integer` | false
 originalPosition | `Integer` | false
 diffHunk | `String` | false
+pullRequestUrl | `String` | false
+pullRequestReviewId | `Long` | false
+inReplyToId | `Long` | false
 
 ### Methods
 > void delete()
@@ -531,7 +534,7 @@ for (comment in pullRequest.comments) {
 ### Listing a Pull Request's review comments
 ```groovy
 for (reviewComment in pullRequest.reviewComments) {
-  echo "File: ${reviewComment.path}, Line: ${reviewComment.line}, Author: ${reviewComment.user}, Comment: ${reviewComment.body}"
+  echo "File: ${reviewComment.path}, Position: ${reviewComment.position}, Author: ${reviewComment.user}, Comment: ${reviewComment.body}"
 }
 ```
 

src/main/java/org/jenkinsci/plugins/pipeline/github/ReviewCommentGroovyObject.java

@@ -43,6 +43,7 @@ public class ReviewCommentGroovyObject extends GroovyObjectSupport implements Se
     }
 
     @Whitelisted
+    @Deprecated
     public Integer getLine() {
         return commitComment.getLine();
     }
@@ -107,6 +108,21 @@ public String getUser() {
         return GitHubHelper.userToLogin(commitComment.getUser());
     }
 
+    @Whitelisted
+    public long getInReplyToId() {
+        return commitComment.getInReplyToId();
+    }
+
+    @Whitelisted
+    public String getPullRequestUrl() {
+        return commitComment.getPullRequestUrl();
+    }
+
+    @Whitelisted
+    public long getPullRequestReviewId() {
+        return commitComment.getPullRequestReviewId();
+    }
+
     @Whitelisted
     public void setBody(final String body) {
         Objects.requireNonNull(body, "body cannot be null");

src/main/java/org/jenkinsci/plugins/pipeline/github/client/ExtendedCommitComment.java

@@ -1,74 +1,38 @@
 package org.jenkinsci.plugins.pipeline.github.client;
 
-import org.eclipse.egit.github.core.Comment;
+import org.eclipse.egit.github.core.CommitComment;
 
 /**
  * @author Aaron Whiteside
  */
-public class ExtendedCommitComment extends Comment {
+public class ExtendedCommitComment extends CommitComment {
     private static final long serialVersionUID = 4834285683963788350L;
 
-    private Integer line;
-    private Integer position;
-    private Integer originalPosition;
-    private String commitId;
-    private String originalCommitId;
-    private String path;
-    private String diffHunk;
+    private String pullRequestUrl;
+    private long pullRequestReviewId;
+    private long inReplyToId;
 
-    public Integer getLine() {
-        return line;
+    public long getInReplyToId() {
+        return inReplyToId;
     }
 
-    public void setLine(final Integer line) {
-        this.line = line;
+    public void setInReplyToId(final long inReplyToId) {
+        this.inReplyToId = inReplyToId;
     }
 
-    public Integer getPosition() {
-        return position;
+    public String getPullRequestUrl() {
+        return pullRequestUrl;
     }
 
-    public void setPosition(final Integer position) {
-        this.position = position;
+    public void setPullRequestUrl(final String pullRequestUrl) {
+        this.pullRequestUrl = pullRequestUrl;
     }
 
-    public Integer getOriginalPosition() {
-        return originalPosition;
+    public long getPullRequestReviewId() {
+        return pullRequestReviewId;
     }
 
-    public void setOriginalPosition(final Integer originalPosition) {
-        this.originalPosition = originalPosition;
-    }
-
-    public String getCommitId() {
-        return commitId;
-    }
-
-    public void setCommitId(final String commitId) {
-        this.commitId = commitId;
-    }
-
-    public String getOriginalCommitId() {
-        return originalCommitId;
-    }
-
-    public void setOriginalCommitId(final String originalCommitId) {
-        this.originalCommitId = originalCommitId;
-    }
-
-    public String getPath() {
-        return path;
-    }
-
-    public void setPath(final String path) {
-        this.path = path;
-    }
-
-    public String getDiffHunk() {
-        return diffHunk;
-    }
-
-    public void setDiffHunk(final String diffHunk) {
-        this.diffHunk = diffHunk;
+    public void setPullRequestReviewId(long pullRequestReviewId) {
+        this.pullRequestReviewId = pullRequestReviewId;
     }
 }