From a8ecf0bef0201f8c344d4e3505e6773afd021fab Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Thu, 12 Apr 2012 11:36:57 -0700
Subject: [PATCH 01/12] [maven-release-plugin] prepare for next development
 iteration

---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 6f2201f303..9cff020515 100644
--- a/pom.xml
+++ b/pom.xml
@@ -7,7 +7,7 @@
   </parent>
 
   <artifactId>github-api</artifactId>
-  <version>1.22</version>
+  <version>1.23-SNAPSHOT</version>
   <name>GitHub API for Java</name>
   <url>http://github-api.kohsuke.org/</url>
   <description>GitHub API for Java</description>

From 53d09bb5d8212c4cb627f929d575eb0e8ce8def3 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Tue, 24 Apr 2012 16:11:55 -0700
Subject: [PATCH 02/12] added object representation for GitHub commit

---
 .../java/org/kohsuke/github/GHCommit.java     | 203 ++++++++++++++++++
 .../java/org/kohsuke/github/GHRepository.java |  13 ++
 2 files changed, 216 insertions(+)
 create mode 100644 src/main/java/org/kohsuke/github/GHCommit.java

diff --git a/src/main/java/org/kohsuke/github/GHCommit.java b/src/main/java/org/kohsuke/github/GHCommit.java
new file mode 100644
index 0000000000..e1a22e5724
--- /dev/null
+++ b/src/main/java/org/kohsuke/github/GHCommit.java
@@ -0,0 +1,203 @@
+package org.kohsuke.github;
+
+import java.io.IOException;
+import java.net.URL;
+import java.util.AbstractList;
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.List;
+
+/**
+ * A commit in a repository.
+ *
+ * @author Kohsuke Kawaguchi
+ * @see GHRepository#getCommit(String)
+ */
+public class GHCommit {
+    private GHRepository owner;
+
+    public static class Stats {
+        int total,additions,deletions;
+    }
+
+    /**
+     * A file that was modified.
+     */
+    public class File {
+        String status;
+        int changes,additions,deletions;
+        String raw_url, blob_url, filename, sha, patch;
+
+        /**
+         * Number of lines added + removed.
+         */
+        public int getLinesChanged() {
+            return changes;
+        }
+
+        /**
+         * Number of lines added.
+         */
+        public int getLinesAdded() {
+            return additions;
+        }
+
+        /**
+         * Number of lines removed.
+         */
+        public int getLinesDeleted() {
+            return deletions;
+        }
+
+        /**
+         * "modified", "added", or "deleted"
+         */
+        public String getStatus() {
+            return status;
+        }
+
+        /**
+         * Just the base name and the extension without any directory name.
+         */
+        public String getFileName() {
+            return filename;
+        }
+
+        /**
+         * The actual change.
+         */
+        public String getPatch() {
+            return patch;
+        }
+
+        /**
+         * URL like 'https://raw.github.com/jenkinsci/jenkins/4eb17c197dfdcf8ef7ff87eb160f24f6a20b7f0e/core/pom.xml'
+         * that resolves to the actual content of the file.
+         */
+        public URL getRawUrl() {
+            return GitHub.parseURL(raw_url);
+        }
+
+        /**
+         * URL like 'https://github.com/jenkinsci/jenkins/blob/1182e2ebb1734d0653142bd422ad33c21437f7cf/core/pom.xml'
+         * that resolves to the HTML page that describes this file.
+         */
+        public URL getBlobUrl() {
+            return GitHub.parseURL(blob_url);
+        }
+
+        /**
+         * [0-9a-f]{40} SHA1 checksum.
+         */
+        public String getSha() {
+            return sha;
+        }
+    }
+
+    public static class Parent {
+        String url,sha;
+    }
+
+    static class User {
+        // TODO: what if someone who doesn't have an account on GitHub makes a commit?
+        String url,avatar_url,login,gravatar_id;
+        int id;
+    }
+
+    String url,sha;
+    List<File> files;
+    Stats stats;
+    List<Parent> parents;
+    User author,committer;
+
+    /**
+     * The repository that contains the commit.
+     */
+    public GHRepository getOwner() {
+        return owner;
+    }
+
+    /**
+     * Number of lines added + removed.
+     */
+    public int getLinesChanged() {
+        return stats.total;
+    }
+
+    /**
+     * Number of lines added.
+     */
+    public int getLinesAdded() {
+        return stats.additions;
+    }
+
+    /**
+     * Number of lines removed.
+     */
+    public int getLinesDeleted() {
+        return stats.deletions;
+    }
+
+    /**
+     * [0-9a-f]{40} SHA1 checksum.
+     */
+    public String getSha() {
+        return sha;
+    }
+
+    /**
+     * List of files changed/added/removed in this commit.
+     *
+     * @return
+     *      Can be empty but never null.
+     */
+    public List<File> getFiles() {
+        return files!=null ? Collections.unmodifiableList(files) : Collections.<File>emptyList();
+    }
+
+    /**
+     * Returns the SHA1 of parent commit objects.
+     */
+    public List<String> getParentShas() {
+        if (parents==null)  return Collections.emptyList();
+        return new AbstractList<String>() {
+            @Override
+            public String get(int index) {
+                return parents.get(index).sha;
+            }
+
+            @Override
+            public int size() {
+                return parents.size();
+            }
+        };
+    }
+
+    /**
+     * Resolves the parent commit objects and return them.
+     */
+    public List<GHCommit> getParents() throws IOException {
+        List<GHCommit> r = new ArrayList<GHCommit>();
+        for (String sha1 : getParentShas())
+            r.add(owner.getCommit(sha1));
+        return r;
+    }
+
+    public GHUser getAuthor() throws IOException {
+        return resolveUser(author);
+    }
+
+    public GHUser getCommitter() throws IOException {
+        return resolveUser(committer);
+    }
+
+    private GHUser resolveUser(User author) throws IOException {
+        if (author==null || author.login==null) return null;
+        return owner.root.getUser(author.login);
+    }
+
+    GHCommit wrapUp(GHRepository owner) {
+        this.owner = owner;
+        return this;
+    }
+}
diff --git a/src/main/java/org/kohsuke/github/GHRepository.java b/src/main/java/org/kohsuke/github/GHRepository.java
index 2a2c4f8327..c203555012 100644
--- a/src/main/java/org/kohsuke/github/GHRepository.java
+++ b/src/main/java/org/kohsuke/github/GHRepository.java
@@ -73,6 +73,7 @@ public class GHRepository {
     private Map<Integer,GHMilestone> milestones = new HashMap<Integer, GHMilestone>();
     
     private String master_branch;
+    private Map<String,GHCommit> commits = new HashMap<String, GHCommit>();
 
     public String getDescription() {
         return description;
@@ -360,6 +361,18 @@ public GHHook getHook(int id) throws IOException {
         return root.retrieveWithAuth3(String.format("/repos/%s/%s/hooks/%d",owner.login,name,id),GHHook.class).wrap(this);
     }
 
+    /**
+     * Gets a commit object in this repository.
+     */
+    public GHCommit getCommit(String sha1) throws IOException {
+        GHCommit c = commits.get(sha1);
+        if (c==null) {
+            c = root.retrieve3(String.format("/repos/%s/%s/commits/%s",owner.login,name,sha1),GHCommit.class).wrapUp(this);
+            commits.put(sha1,c);
+        }
+        return c;
+    }
+
     /**
      * 
      * See https://api.github.com/hooks for possible names and their configuration scheme.

From fff07bf70b43f3eb2bda74aee114860aed960c14 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Tue, 24 Apr 2012 16:14:13 -0700
Subject: [PATCH 03/12] fixed a bug in error handling

---
 src/main/java/org/kohsuke/github/GitHub.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/java/org/kohsuke/github/GitHub.java b/src/main/java/org/kohsuke/github/GitHub.java
index 91b34b1d7a..faffd27d79 100644
--- a/src/main/java/org/kohsuke/github/GitHub.java
+++ b/src/main/java/org/kohsuke/github/GitHub.java
@@ -320,7 +320,7 @@ private <T> T parse(HttpURLConnection uc, Class<T> type) throws IOException {
      */
     private InputStream wrapStream(HttpURLConnection uc, InputStream in) throws IOException {
         String encoding = uc.getContentEncoding();
-        if (encoding==null) return in;
+        if (encoding==null || in==null) return in;
         if (encoding.equals("gzip"))    return new GZIPInputStream(in);
 
         throw new UnsupportedOperationException("Unexpected Content-Encoding: "+encoding);

From a124d7f7143613ab19f4d6af0428ceed59a2a614 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Tue, 24 Apr 2012 16:16:13 -0700
Subject: [PATCH 04/12] added some tests

---
 src/main/java/org/kohsuke/github/GHCommit.java |  2 +-
 src/test/java/org/kohsuke/AppTest.java         | 15 +++++++++++++++
 2 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/src/main/java/org/kohsuke/github/GHCommit.java b/src/main/java/org/kohsuke/github/GHCommit.java
index e1a22e5724..7acf118788 100644
--- a/src/main/java/org/kohsuke/github/GHCommit.java
+++ b/src/main/java/org/kohsuke/github/GHCommit.java
@@ -23,7 +23,7 @@ public static class Stats {
     /**
      * A file that was modified.
      */
-    public class File {
+    public static class File {
         String status;
         int changes,additions,deletions;
         String raw_url, blob_url, filename, sha, patch;
diff --git a/src/test/java/org/kohsuke/AppTest.java b/src/test/java/org/kohsuke/AppTest.java
index aabee36d41..02598d0d04 100644
--- a/src/test/java/org/kohsuke/AppTest.java
+++ b/src/test/java/org/kohsuke/AppTest.java
@@ -1,6 +1,8 @@
 package org.kohsuke;
 
 import junit.framework.TestCase;
+import org.kohsuke.github.GHCommit;
+import org.kohsuke.github.GHCommit.File;
 import org.kohsuke.github.GHEvent;
 import org.kohsuke.github.GHEventInfo;
 import org.kohsuke.github.GHEventPayload;
@@ -74,6 +76,19 @@ public void testMemberOrgs() throws Exception {
         System.out.println(o);
     }
 
+    public void testCommit() throws Exception {
+        GitHub gitHub = GitHub.connect();
+        GHCommit commit = gitHub.getUser("jenkinsci").getRepository("jenkins").getCommit("08c1c9970af4d609ae754fbe803e06186e3206f7");
+        System.out.println(commit);
+        assertEquals(1, commit.getParents().size());
+        assertEquals(1,commit.getFiles().size());
+
+        File f = commit.getFiles().get(0);
+        assertEquals(48,f.getLinesChanged());
+        assertEquals("modified",f.getStatus());
+        assertEquals("changelog.html",f.getFileName());
+    }
+
     public void testBranches() throws Exception {
         GitHub gitHub = GitHub.connect();
         Map<String,GHBranch> b =

From 39631461ae66be7da0dd2d4879279e569557976b Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Tue, 24 Apr 2012 16:30:01 -0700
Subject: [PATCH 05/12] added list commits

---
 .../java/org/kohsuke/github/GHRepository.java | 13 +++-
 .../org/kohsuke/github/PagedIterable.java     |  8 +++
 .../org/kohsuke/github/PagedIterator.java     | 61 +++++++++++++++++++
 src/test/java/org/kohsuke/AppTest.java        | 12 ++++
 4 files changed, 93 insertions(+), 1 deletion(-)
 create mode 100644 src/main/java/org/kohsuke/github/PagedIterable.java
 create mode 100644 src/main/java/org/kohsuke/github/PagedIterator.java

diff --git a/src/main/java/org/kohsuke/github/GHRepository.java b/src/main/java/org/kohsuke/github/GHRepository.java
index c203555012..3a1144b056 100644
--- a/src/main/java/org/kohsuke/github/GHRepository.java
+++ b/src/main/java/org/kohsuke/github/GHRepository.java
@@ -272,7 +272,7 @@ public void delete() throws IOException {
         String url = "/repos/delete/" + owner.login +"/"+name;
 
         DeleteToken token = poster.to(url, DeleteToken.class);
-        poster.with("delete_token",token.delete_token).to(url);
+        poster.with("delete_token", token.delete_token).to(url);
     }
 
     /**
@@ -373,6 +373,17 @@ public GHCommit getCommit(String sha1) throws IOException {
         return c;
     }
 
+    /**
+     * Lists all the commits.
+     */
+    public PagedIterable<GHCommit> listCommits() {
+        return new PagedIterable<GHCommit>() {
+            public PagedIterator<GHCommit> iterator() {
+                return new PagedIterator<GHCommit>(root.retrievePaged(String.format("/repos/%s/%s/commits",owner.login,name),GHCommit[].class,false,V3));
+            }
+        };
+    }
+
     /**
      * 
      * See https://api.github.com/hooks for possible names and their configuration scheme.
diff --git a/src/main/java/org/kohsuke/github/PagedIterable.java b/src/main/java/org/kohsuke/github/PagedIterable.java
new file mode 100644
index 0000000000..2470ca7297
--- /dev/null
+++ b/src/main/java/org/kohsuke/github/PagedIterable.java
@@ -0,0 +1,8 @@
+package org.kohsuke.github;
+
+/**
+ * @author Kohsuke Kawaguchi
+ */
+public interface PagedIterable<T> extends Iterable<T> {
+    PagedIterator<T> iterator();
+}
diff --git a/src/main/java/org/kohsuke/github/PagedIterator.java b/src/main/java/org/kohsuke/github/PagedIterator.java
new file mode 100644
index 0000000000..7e84588ddf
--- /dev/null
+++ b/src/main/java/org/kohsuke/github/PagedIterator.java
@@ -0,0 +1,61 @@
+package org.kohsuke.github;
+
+import java.util.Arrays;
+import java.util.Iterator;
+import java.util.List;
+
+/**
+ * Iterator over a pagenated data source.
+ *
+ * Aside from the normal iterator operation, this method exposes {@link #nextPage()}
+ * that allows the caller to retrieve items per page.
+ *
+ * @author Kohsuke Kawaguchi
+ */
+public class PagedIterator<T> implements Iterator<T> {
+    private final Iterator<T[]> base;
+
+    /**
+     * Current batch that we retrieved but haven't returned to the caller.
+     */
+    private T[] current;
+    private int pos;
+
+    public PagedIterator(Iterator<T[]> base) {
+        this.base = base;
+    }
+
+    public boolean hasNext() {
+        return (current!=null && pos<current.length) || base.hasNext();
+    }
+
+    public T next() {
+        fetch();
+
+        return current[pos++];
+    }
+
+    private void fetch() {
+        while (current==null || current.length<=pos) {
+            current = base.next();
+            pos = 0;
+        }
+        // invariant at the end: there's some data to retrieve
+    }
+
+    public void remove() {
+        throw new UnsupportedOperationException();
+    }
+
+    /**
+     * Gets the next page worth of data.
+     */
+    public List<T> nextPage() {
+        fetch();
+        List<T> r = Arrays.asList(current);
+        r = r.subList(pos,r.size());
+        current = null;
+        pos = 0;
+        return r;
+    }
+}
diff --git a/src/test/java/org/kohsuke/AppTest.java b/src/test/java/org/kohsuke/AppTest.java
index 02598d0d04..dac2ea0b0f 100644
--- a/src/test/java/org/kohsuke/AppTest.java
+++ b/src/test/java/org/kohsuke/AppTest.java
@@ -19,6 +19,7 @@
 
 import java.io.IOException;
 import java.net.URL;
+import java.util.ArrayList;
 import java.util.Map;
 import java.util.Set;
 import java.util.List;
@@ -89,6 +90,17 @@ public void testCommit() throws Exception {
         assertEquals("changelog.html",f.getFileName());
     }
 
+    public void testListCommits() throws Exception {
+        GitHub gitHub = GitHub.connect();
+        List<String> sha1 = new ArrayList<String>();
+        for (GHCommit c : gitHub.getUser("kohsuke").getRepository("empty-commit").listCommits()) {
+            System.out.println(c.getSha());
+            sha1.add(c.getSha());
+        }
+        assertEquals("fdfad6be4db6f96faea1f153fb447b479a7a9cb7",sha1.get(0));
+        assertEquals(1,sha1.size());
+    }
+
     public void testBranches() throws Exception {
         GitHub gitHub = GitHub.connect();
         Map<String,GHBranch> b =

From 8557676561e5d8c5daa7a213901381de03a5a269 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Tue, 24 Apr 2012 16:36:18 -0700
Subject: [PATCH 06/12] making names consistent with GHBranch

---
 src/main/java/org/kohsuke/github/GHCommit.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/main/java/org/kohsuke/github/GHCommit.java b/src/main/java/org/kohsuke/github/GHCommit.java
index 7acf118788..4454515cec 100644
--- a/src/main/java/org/kohsuke/github/GHCommit.java
+++ b/src/main/java/org/kohsuke/github/GHCommit.java
@@ -141,7 +141,7 @@ public int getLinesDeleted() {
     /**
      * [0-9a-f]{40} SHA1 checksum.
      */
-    public String getSha() {
+    public String getSHA1() {
         return sha;
     }
 
@@ -158,7 +158,7 @@ public List<File> getFiles() {
     /**
      * Returns the SHA1 of parent commit objects.
      */
-    public List<String> getParentShas() {
+    public List<String> getParentSHA1s() {
         if (parents==null)  return Collections.emptyList();
         return new AbstractList<String>() {
             @Override
@@ -178,7 +178,7 @@ public int size() {
      */
     public List<GHCommit> getParents() throws IOException {
         List<GHCommit> r = new ArrayList<GHCommit>();
-        for (String sha1 : getParentShas())
+        for (String sha1 : getParentSHA1s())
             r.add(owner.getCommit(sha1));
         return r;
     }

From 31a6eca97f1235c12f2fb96e4ab6c780de5362d7 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Tue, 24 Apr 2012 16:54:05 -0700
Subject: [PATCH 07/12] added commit comment support

---
 .../java/org/kohsuke/github/GHCommit.java     |  20 ++++
 .../org/kohsuke/github/GHCommitComment.java   | 100 ++++++++++++++++++
 .../java/org/kohsuke/github/GHPerson.java     |   2 +
 .../java/org/kohsuke/github/GHRepository.java |  24 ++++-
 .../org/kohsuke/github/PagedIterator.java     |   6 +-
 src/test/java/org/kohsuke/AppTest.java        |  17 ++-
 6 files changed, 164 insertions(+), 5 deletions(-)
 create mode 100644 src/main/java/org/kohsuke/github/GHCommitComment.java

diff --git a/src/main/java/org/kohsuke/github/GHCommit.java b/src/main/java/org/kohsuke/github/GHCommit.java
index 4454515cec..dd91c118cf 100644
--- a/src/main/java/org/kohsuke/github/GHCommit.java
+++ b/src/main/java/org/kohsuke/github/GHCommit.java
@@ -7,11 +7,14 @@
 import java.util.Collections;
 import java.util.List;
 
+import static org.kohsuke.github.ApiVersion.V3;
+
 /**
  * A commit in a repository.
  *
  * @author Kohsuke Kawaguchi
  * @see GHRepository#getCommit(String)
+ * @see GHCommitComment#getCommit()
  */
 public class GHCommit {
     private GHRepository owner;
@@ -196,6 +199,23 @@ private GHUser resolveUser(User author) throws IOException {
         return owner.root.getUser(author.login);
     }
 
+    /**
+     * Lists up all the commit comments in this repository.
+     */
+    public PagedIterable<GHCommitComment> listComments() {
+        return new PagedIterable<GHCommitComment>() {
+            public PagedIterator<GHCommitComment> iterator() {
+                return new PagedIterator<GHCommitComment>(owner.root.retrievePaged(String.format("/repos/%s/%s/commits/%s/comments",owner.getOwnerName(),owner.getName(),sha),GHCommitComment[].class,false,V3)) {
+                    @Override
+                    protected void wrapUp(GHCommitComment[] page) {
+                        for (GHCommitComment c : page)
+                            c.wrap(owner);
+                    }
+                };
+            }
+        };
+    }
+
     GHCommit wrapUp(GHRepository owner) {
         this.owner = owner;
         return this;
diff --git a/src/main/java/org/kohsuke/github/GHCommitComment.java b/src/main/java/org/kohsuke/github/GHCommitComment.java
new file mode 100644
index 0000000000..7b44fa3db2
--- /dev/null
+++ b/src/main/java/org/kohsuke/github/GHCommitComment.java
@@ -0,0 +1,100 @@
+package org.kohsuke.github;
+
+import java.io.IOException;
+import java.net.URL;
+import java.util.Date;
+
+/**
+ * A comment attached to a commit (or a specific line in a specific file of a commit.)
+ *
+ * @author Kohsuke Kawaguchi
+ * @see GHRepository#listCommitComments()
+ * @see GHCommit#listComments()
+ */
+public class GHCommitComment {
+    private GHRepository owner;
+
+    String updated_at, created_at;
+    String body, url, html_url, commit_id;
+    Integer line;
+    int id;
+    String path;
+    User user;
+
+    static class User {
+        // TODO: what if someone who doesn't have an account on GitHub makes a commit?
+        String url,avatar_url,login,gravatar_id;
+        int id;
+    }
+
+    public GHRepository getOwner() {
+        return owner;
+    }
+
+    public Date getCreatedAt() {
+        return GitHub.parseDate(created_at);
+    }
+
+    public Date getUpdatedAt() {
+        return GitHub.parseDate(updated_at);
+    }
+
+    /**
+     * URL like 'https://github.com/kohsuke/sandbox-ant/commit/8ae38db0ea5837313ab5f39d43a6f73de3bd9000#commitcomment-1252827' to
+     * show this commit comment in a browser.
+     */
+    public URL getHtmlUrl() {
+        return GitHub.parseURL(html_url);
+    }
+
+    public String getSHA1() {
+        return commit_id;
+    }
+
+    /**
+     * Commit comment in the GitHub flavored markdown format.
+     */
+    public String getBody() {
+        return body;
+    }
+
+    /**
+     * A commit comment can be on a specific line of a specific file, if so, this field points to a file.
+     * Otherwise null.
+     */
+    public String getPath() {
+        return path;
+    }
+
+    /**
+     * A commit comment can be on a specific line of a specific file, if so, this field points to the line number in the file.
+     * Otherwise -1.
+     */
+    public int getLine() {
+        return line!=null ? line : -1;
+    }
+
+    public int getId() {
+        return id;
+    }
+
+    /**
+     * Gets the user who put this comment.
+     */
+    public GHUser getUser() throws IOException {
+        return owner.root.getUser(user.login);
+    }
+
+    /**
+     * Gets the commit to which this comment is associated with.
+     */
+    public GHCommit getCommit() throws IOException {
+        return getOwner().getCommit(getSHA1());
+    }
+
+
+    GHCommitComment wrap(GHRepository owner) {
+        this.owner = owner;
+        return this;
+    }
+}
diff --git a/src/main/java/org/kohsuke/github/GHPerson.java b/src/main/java/org/kohsuke/github/GHPerson.java
index 5a03ed9451..70ba8f27d0 100644
--- a/src/main/java/org/kohsuke/github/GHPerson.java
+++ b/src/main/java/org/kohsuke/github/GHPerson.java
@@ -1,5 +1,7 @@
 package org.kohsuke.github;
 
+import com.infradna.tool.bridge_method_injector.WithBridgeMethods;
+
 import java.io.FileNotFoundException;
 import java.io.IOException;
 import java.util.Arrays;
diff --git a/src/main/java/org/kohsuke/github/GHRepository.java b/src/main/java/org/kohsuke/github/GHRepository.java
index 3a1144b056..909554a6b0 100644
--- a/src/main/java/org/kohsuke/github/GHRepository.java
+++ b/src/main/java/org/kohsuke/github/GHRepository.java
@@ -379,7 +379,29 @@ public GHCommit getCommit(String sha1) throws IOException {
     public PagedIterable<GHCommit> listCommits() {
         return new PagedIterable<GHCommit>() {
             public PagedIterator<GHCommit> iterator() {
-                return new PagedIterator<GHCommit>(root.retrievePaged(String.format("/repos/%s/%s/commits",owner.login,name),GHCommit[].class,false,V3));
+                return new PagedIterator<GHCommit>(root.retrievePaged(String.format("/repos/%s/%s/commits",owner.login,name),GHCommit[].class,false,V3)) {
+                    protected void wrapUp(GHCommit[] page) {
+                        for (GHCommit c : page)
+                            c.wrapUp(GHRepository.this);
+                    }
+                };
+            }
+        };
+    }
+
+    /**
+     * Lists up all the commit comments in this repository.
+     */
+    public PagedIterable<GHCommitComment> listCommitComments() {
+        return new PagedIterable<GHCommitComment>() {
+            public PagedIterator<GHCommitComment> iterator() {
+                return new PagedIterator<GHCommitComment>(root.retrievePaged(String.format("/repos/%s/%s/comments",owner.login,name),GHCommitComment[].class,false,V3)) {
+                    @Override
+                    protected void wrapUp(GHCommitComment[] page) {
+                        for (GHCommitComment c : page)
+                            c.wrap(GHRepository.this);
+                    }
+                };
             }
         };
     }
diff --git a/src/main/java/org/kohsuke/github/PagedIterator.java b/src/main/java/org/kohsuke/github/PagedIterator.java
index 7e84588ddf..dc670a16d5 100644
--- a/src/main/java/org/kohsuke/github/PagedIterator.java
+++ b/src/main/java/org/kohsuke/github/PagedIterator.java
@@ -12,7 +12,7 @@
  *
  * @author Kohsuke Kawaguchi
  */
-public class PagedIterator<T> implements Iterator<T> {
+public abstract class PagedIterator<T> implements Iterator<T> {
     private final Iterator<T[]> base;
 
     /**
@@ -21,10 +21,12 @@ public class PagedIterator<T> implements Iterator<T> {
     private T[] current;
     private int pos;
 
-    public PagedIterator(Iterator<T[]> base) {
+    /*package*/ PagedIterator(Iterator<T[]> base) {
         this.base = base;
     }
 
+    protected abstract void wrapUp(T[] page);
+
     public boolean hasNext() {
         return (current!=null && pos<current.length) || base.hasNext();
     }
diff --git a/src/test/java/org/kohsuke/AppTest.java b/src/test/java/org/kohsuke/AppTest.java
index dac2ea0b0f..f09664dda3 100644
--- a/src/test/java/org/kohsuke/AppTest.java
+++ b/src/test/java/org/kohsuke/AppTest.java
@@ -3,6 +3,7 @@
 import junit.framework.TestCase;
 import org.kohsuke.github.GHCommit;
 import org.kohsuke.github.GHCommit.File;
+import org.kohsuke.github.GHCommitComment;
 import org.kohsuke.github.GHEvent;
 import org.kohsuke.github.GHEventInfo;
 import org.kohsuke.github.GHEventPayload;
@@ -16,6 +17,7 @@
 import org.kohsuke.github.GHTeam;
 import org.kohsuke.github.GHUser;
 import org.kohsuke.github.GitHub;
+import org.kohsuke.github.PagedIterable;
 
 import java.io.IOException;
 import java.net.URL;
@@ -94,8 +96,8 @@ public void testListCommits() throws Exception {
         GitHub gitHub = GitHub.connect();
         List<String> sha1 = new ArrayList<String>();
         for (GHCommit c : gitHub.getUser("kohsuke").getRepository("empty-commit").listCommits()) {
-            System.out.println(c.getSha());
-            sha1.add(c.getSha());
+            System.out.println(c.getSHA1());
+            sha1.add(c.getSHA1());
         }
         assertEquals("fdfad6be4db6f96faea1f153fb447b479a7a9cb7",sha1.get(0));
         assertEquals(1,sha1.size());
@@ -107,6 +109,17 @@ public void testBranches() throws Exception {
                 gitHub.getUser("jenkinsci").getRepository("jenkins").getBranches();
         System.out.println(b);
     }
+
+    public void testCommitComment() throws Exception {
+        GitHub gitHub = GitHub.connect();
+        GHRepository r = gitHub.getUser("jenkinsci").getRepository("jenkins");
+        PagedIterable<GHCommitComment> comments = r.listCommitComments();
+        List<GHCommitComment> batch = comments.iterator().nextPage();
+        for (GHCommitComment comment : batch) {
+            System.out.println(comment.getBody());
+            assertSame(comment.getOwner(),r);
+        }
+    }
     
     public void tryHook() throws Exception {
         GitHub gitHub = GitHub.connect();

From b5514b891ac558d9903ce4872b9c49d4850fabe3 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Tue, 24 Apr 2012 17:06:23 -0700
Subject: [PATCH 08/12] added a method to create a new commit comment

---
 .../java/org/kohsuke/github/GHCommit.java     | 20 +++++++++++++++++++
 .../org/kohsuke/github/GHCommitComment.java   |  1 +
 src/main/java/org/kohsuke/github/Poster.java  |  6 ++++++
 src/test/java/org/kohsuke/AppTest.java        | 11 ++++++++--
 4 files changed, 36 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/kohsuke/github/GHCommit.java b/src/main/java/org/kohsuke/github/GHCommit.java
index dd91c118cf..9ba4ca2bd8 100644
--- a/src/main/java/org/kohsuke/github/GHCommit.java
+++ b/src/main/java/org/kohsuke/github/GHCommit.java
@@ -216,6 +216,26 @@ protected void wrapUp(GHCommitComment[] page) {
         };
     }
 
+    /**
+     * Creates a commit comment.
+     *
+     * I'm not sure how path/line/position parameters interact with each other.
+     */
+    public GHCommitComment createComment(String body, String path, Integer line, Integer position) throws IOException {
+        GHCommitComment r = new Poster(owner.root,V3)
+                .with("body",body)
+                .with("path",path)
+                .with("line",line)
+                .with("position",position)
+                .withCredential()
+                .to(String.format("/repos/%s/%s/commits/%s/comments",owner.getOwnerName(),owner.getName(),sha),GHCommitComment.class);
+        return r.wrap(owner);
+    }
+
+    public GHCommitComment createComment(String body) throws IOException {
+        return createComment(body,null,null,null);
+    }
+
     GHCommit wrapUp(GHRepository owner) {
         this.owner = owner;
         return this;
diff --git a/src/main/java/org/kohsuke/github/GHCommitComment.java b/src/main/java/org/kohsuke/github/GHCommitComment.java
index 7b44fa3db2..f5137e7fdd 100644
--- a/src/main/java/org/kohsuke/github/GHCommitComment.java
+++ b/src/main/java/org/kohsuke/github/GHCommitComment.java
@@ -10,6 +10,7 @@
  * @author Kohsuke Kawaguchi
  * @see GHRepository#listCommitComments()
  * @see GHCommit#listComments()
+ * @see GHCommit#createComment(String, String, Integer, Integer)
  */
 public class GHCommitComment {
     private GHRepository owner;
diff --git a/src/main/java/org/kohsuke/github/Poster.java b/src/main/java/org/kohsuke/github/Poster.java
index 556044a986..128d2f6e8d 100644
--- a/src/main/java/org/kohsuke/github/Poster.java
+++ b/src/main/java/org/kohsuke/github/Poster.java
@@ -86,6 +86,12 @@ public Poster with(String key, int value) {
         return _with(key, value);
     }
 
+    public Poster with(String key, Integer value) {
+        if (value!=null)
+            _with(key, value.intValue());
+        return this;
+    }
+
     public Poster with(String key, boolean value) {
         return _with(key, value);
     }
diff --git a/src/test/java/org/kohsuke/AppTest.java b/src/test/java/org/kohsuke/AppTest.java
index f09664dda3..e06ada4d9c 100644
--- a/src/test/java/org/kohsuke/AppTest.java
+++ b/src/test/java/org/kohsuke/AppTest.java
@@ -117,10 +117,17 @@ public void testCommitComment() throws Exception {
         List<GHCommitComment> batch = comments.iterator().nextPage();
         for (GHCommitComment comment : batch) {
             System.out.println(comment.getBody());
-            assertSame(comment.getOwner(),r);
+            assertSame(comment.getOwner(), r);
         }
     }
-    
+
+    public void tryCreateCommitComment() throws Exception {
+        GitHub gitHub = GitHub.connect();
+        GHCommit commit = gitHub.getUser("kohsuke").getRepository("sandbox-ant").getCommit("8ae38db0ea5837313ab5f39d43a6f73de3bd9000");
+        GHCommitComment c = commit.createComment("[testing](http://kohsuse.org/)");
+        System.out.println(c);
+    }
+
     public void tryHook() throws Exception {
         GitHub gitHub = GitHub.connect();
         GHRepository r = gitHub.getMyself().getRepository("test2");

From dd54a00172921edba224600fa91209a36e5b3e3f Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Tue, 24 Apr 2012 17:13:48 -0700
Subject: [PATCH 09/12] added update

---
 .../java/org/kohsuke/github/GHCommitComment.java  | 13 +++++++++++++
 src/main/java/org/kohsuke/github/Poster.java      | 15 ++++++++++++++-
 src/test/java/org/kohsuke/AppTest.java            |  4 +++-
 3 files changed, 30 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/kohsuke/github/GHCommitComment.java b/src/main/java/org/kohsuke/github/GHCommitComment.java
index f5137e7fdd..5151e119d3 100644
--- a/src/main/java/org/kohsuke/github/GHCommitComment.java
+++ b/src/main/java/org/kohsuke/github/GHCommitComment.java
@@ -4,6 +4,8 @@
 import java.net.URL;
 import java.util.Date;
 
+import static org.kohsuke.github.ApiVersion.V3;
+
 /**
  * A comment attached to a commit (or a specific line in a specific file of a commit.)
  *
@@ -93,6 +95,17 @@ public GHCommit getCommit() throws IOException {
         return getOwner().getCommit(getSHA1());
     }
 
+    /**
+     * Updates the body of the commit message.
+     */
+    public void update(String body) throws IOException {
+        GHCommitComment r = new Poster(owner.root,V3)
+                .with("body",body)
+                .withCredential()
+                .to(String.format("/repos/%s/%s/comments/%s",owner.getOwnerName(),owner.getName(),id),GHCommitComment.class,"PATCH");
+        this.body = body;
+    }
+
 
     GHCommitComment wrap(GHRepository owner) {
         this.owner = owner;
diff --git a/src/main/java/org/kohsuke/github/Poster.java b/src/main/java/org/kohsuke/github/Poster.java
index 128d2f6e8d..c07b5ddad3 100644
--- a/src/main/java/org/kohsuke/github/Poster.java
+++ b/src/main/java/org/kohsuke/github/Poster.java
@@ -33,7 +33,9 @@
 import java.io.OutputStreamWriter;
 import java.io.Reader;
 import java.io.UnsupportedEncodingException;
+import java.lang.reflect.Field;
 import java.net.HttpURLConnection;
+import java.net.ProtocolException;
 import java.net.URLEncoder;
 import java.util.ArrayList;
 import java.util.HashMap;
@@ -142,7 +144,18 @@ public <T> T to(String tailApiUrl, Class<T> type, String method) throws IOExcept
                     uc.setRequestProperty("Authorization", "Basic " + root.encodedAuthorization);
                 }
             }
-            uc.setRequestMethod(method);
+            try {
+                uc.setRequestMethod(method);
+            } catch (ProtocolException e) {
+                // JDK only allows one of the fixed set of verbs. Try to override that
+                try {
+                    Field $method = HttpURLConnection.class.getDeclaredField("method");
+                    $method.setAccessible(true);
+                    $method.set(uc,method);
+                } catch (Exception x) {
+                    throw (IOException)new IOException("Failed to set the custom verb").initCause(x);
+                }
+            }
 
 
             if (v==ApiVersion.V2) {
diff --git a/src/test/java/org/kohsuke/AppTest.java b/src/test/java/org/kohsuke/AppTest.java
index e06ada4d9c..f092d1960a 100644
--- a/src/test/java/org/kohsuke/AppTest.java
+++ b/src/test/java/org/kohsuke/AppTest.java
@@ -121,11 +121,13 @@ public void testCommitComment() throws Exception {
         }
     }
 
-    public void tryCreateCommitComment() throws Exception {
+    public void testCreateCommitComment() throws Exception {
         GitHub gitHub = GitHub.connect();
         GHCommit commit = gitHub.getUser("kohsuke").getRepository("sandbox-ant").getCommit("8ae38db0ea5837313ab5f39d43a6f73de3bd9000");
         GHCommitComment c = commit.createComment("[testing](http://kohsuse.org/)");
         System.out.println(c);
+        c.update("updated text");
+        System.out.println(c);
     }
 
     public void tryHook() throws Exception {

From 0b69743792731e197e6e3f993aea93b2b98823de Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Tue, 24 Apr 2012 17:17:35 -0700
Subject: [PATCH 10/12] added comment deletion

---
 .../java/org/kohsuke/github/GHCommitComment.java    | 13 ++++++++++++-
 src/test/java/org/kohsuke/AppTest.java              |  3 ++-
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/kohsuke/github/GHCommitComment.java b/src/main/java/org/kohsuke/github/GHCommitComment.java
index 5151e119d3..c2364a23d5 100644
--- a/src/main/java/org/kohsuke/github/GHCommitComment.java
+++ b/src/main/java/org/kohsuke/github/GHCommitComment.java
@@ -102,10 +102,21 @@ public void update(String body) throws IOException {
         GHCommitComment r = new Poster(owner.root,V3)
                 .with("body",body)
                 .withCredential()
-                .to(String.format("/repos/%s/%s/comments/%s",owner.getOwnerName(),owner.getName(),id),GHCommitComment.class,"PATCH");
+                .to(getApiTail(),GHCommitComment.class,"PATCH");
         this.body = body;
     }
 
+    /**
+     * Deletes this comment.
+     */
+    public void delete() throws IOException {
+        new Poster(owner.root,V3).withCredential().to(getApiTail(),null,"DELETE");
+    }
+
+    private String getApiTail() {
+        return String.format("/repos/%s/%s/comments/%s",owner.getOwnerName(),owner.getName(),id);
+    }
+
 
     GHCommitComment wrap(GHRepository owner) {
         this.owner = owner;
diff --git a/src/test/java/org/kohsuke/AppTest.java b/src/test/java/org/kohsuke/AppTest.java
index f092d1960a..1e016e771c 100644
--- a/src/test/java/org/kohsuke/AppTest.java
+++ b/src/test/java/org/kohsuke/AppTest.java
@@ -121,13 +121,14 @@ public void testCommitComment() throws Exception {
         }
     }
 
-    public void testCreateCommitComment() throws Exception {
+    public void tryCreateCommitComment() throws Exception {
         GitHub gitHub = GitHub.connect();
         GHCommit commit = gitHub.getUser("kohsuke").getRepository("sandbox-ant").getCommit("8ae38db0ea5837313ab5f39d43a6f73de3bd9000");
         GHCommitComment c = commit.createComment("[testing](http://kohsuse.org/)");
         System.out.println(c);
         c.update("updated text");
         System.out.println(c);
+        c.delete();
     }
 
     public void tryHook() throws Exception {

From 706cdac4cc63f68ed8636b04b9edc89546388f5c Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Tue, 24 Apr 2012 17:21:33 -0700
Subject: [PATCH 11/12] bug fix

---
 src/main/java/org/kohsuke/github/PagedIterator.java | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/main/java/org/kohsuke/github/PagedIterator.java b/src/main/java/org/kohsuke/github/PagedIterator.java
index dc670a16d5..74c197aa2e 100644
--- a/src/main/java/org/kohsuke/github/PagedIterator.java
+++ b/src/main/java/org/kohsuke/github/PagedIterator.java
@@ -40,6 +40,7 @@ public T next() {
     private void fetch() {
         while (current==null || current.length<=pos) {
             current = base.next();
+            wrapUp(current);
             pos = 0;
         }
         // invariant at the end: there's some data to retrieve

From 86543c84db7d0fd3a3f00a319187dc6dd64c0784 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Tue, 24 Apr 2012 17:22:14 -0700
Subject: [PATCH 12/12] [maven-release-plugin] prepare release github-api-1.23

---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 9cff020515..e0b19ded3d 100644
--- a/pom.xml
+++ b/pom.xml
@@ -7,7 +7,7 @@
   </parent>
 
   <artifactId>github-api</artifactId>
-  <version>1.23-SNAPSHOT</version>
+  <version>1.23</version>
   <name>GitHub API for Java</name>
   <url>http://github-api.kohsuke.org/</url>
   <description>GitHub API for Java</description>