From 18696fca2dcc06314a8d4ae2a49b81180ef35130 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sat, 14 Feb 2015 10:28:55 -0800
Subject: [PATCH 01/19] [maven-release-plugin] prepare for next development
 iteration

---
 pom.xml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/pom.xml b/pom.xml
index c16a726dee..36c9f040a2 100644
--- a/pom.xml
+++ b/pom.xml
@@ -7,7 +7,7 @@
   </parent>
 
   <artifactId>github-api</artifactId>
-  <version>1.61</version>
+  <version>1.62-SNAPSHOT</version>
   <name>GitHub API for Java</name>
   <url>http://github-api.kohsuke.org/</url>
   <description>GitHub API for Java</description>
@@ -16,7 +16,7 @@
     <connection>scm:git:git@github.com/kohsuke/${project.artifactId}.git</connection>
     <developerConnection>scm:git:ssh://git@github.com/kohsuke/${project.artifactId}.git</developerConnection>
     <url>http://${project.artifactId}.kohsuke.org/</url>
-    <tag>github-api-1.61</tag>
+    <tag>HEAD</tag>
   </scm>
 
   <distributionManagement>

From 1dbcc4b7765950d90194775db4ffe4f076f80ea7 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sun, 15 Feb 2015 06:31:22 -0800
Subject: [PATCH 02/19] Fixed the getReadme() method.

It was calling the wrong endpoint.
Fixed issue #99.
---
 src/main/java/org/kohsuke/github/GHGist.java       | 1 +
 src/main/java/org/kohsuke/github/GHRepository.java | 8 ++++++--
 src/test/java/org/kohsuke/github/AppTest.java      | 7 +++++++
 3 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/kohsuke/github/GHGist.java b/src/main/java/org/kohsuke/github/GHGist.java
index 6e3f35d989..801aef22d7 100644
--- a/src/main/java/org/kohsuke/github/GHGist.java
+++ b/src/main/java/org/kohsuke/github/GHGist.java
@@ -15,6 +15,7 @@
  * @see GHUser#listGists()
  * @see GitHub#getGist(String)
  * @see GitHub#createGist()
+ * @see <a href="https://developer.github.com/v3/gists/">documentation</a>
  */
 public class GHGist extends GHObject {
     /*package almost final*/ GHUser owner;
diff --git a/src/main/java/org/kohsuke/github/GHRepository.java b/src/main/java/org/kohsuke/github/GHRepository.java
index 599c09cf25..c0f36153c2 100644
--- a/src/main/java/org/kohsuke/github/GHRepository.java
+++ b/src/main/java/org/kohsuke/github/GHRepository.java
@@ -964,8 +964,12 @@ public List<GHContent> getDirectoryContent(String path, String ref) throws IOExc
         return Arrays.asList(files);
     }
 
-    public GHContent getReadme() throws Exception {
-        return getFileContent("readme");
+    /**
+     * https://developer.github.com/v3/repos/contents/#get-the-readme
+     */
+    public GHContent getReadme() throws IOException {
+        Requester requester = root.retrieve();
+        return requester.to(getApiTailUrl("readme"), GHContent.class).wrap(this);
     }
 
     public GHContentUpdateResponse createContent(String content, String commitMessage, String path) throws IOException {
diff --git a/src/test/java/org/kohsuke/github/AppTest.java b/src/test/java/org/kohsuke/github/AppTest.java
index 65074716fb..3afb464618 100755
--- a/src/test/java/org/kohsuke/github/AppTest.java
+++ b/src/test/java/org/kohsuke/github/AppTest.java
@@ -658,6 +658,13 @@ public void testIssueSearch() throws IOException {
         }
     }
 
+    @Test   // issue #99
+    public void testReadme() throws IOException {
+        GHContent readme = gitHub.getRepository("github-api-test-org/test-readme").getReadme();
+        assertEquals(readme.getName(),"README.md");
+        assertEquals(readme.getContent(),"This is a markdown readme.\n");
+    }
+
     private void kohsuke() {
         String login = getUser().getLogin();
         Assume.assumeTrue(login.equals("kohsuke") || login.equals("kohsuke2"));

From d90adfa98e61d6ce5d7e585ca2f5b2c9f782bbd9 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sun, 15 Feb 2015 06:55:35 -0800
Subject: [PATCH 03/19] Implemented label CRUD operations on GHRepository

Fixes issue #105
---
 src/main/java/org/kohsuke/github/GHIssue.java | 26 ++++---------
 src/main/java/org/kohsuke/github/GHLabel.java | 37 +++++++++++++++++++
 .../org/kohsuke/github/GHPullRequest.java     |  2 +-
 .../java/org/kohsuke/github/GHRepository.java | 30 +++++++++++++++
 src/test/java/org/kohsuke/github/AppTest.java | 25 +++++++++++++
 .../org/kohsuke/github/PullRequestTest.java   |  2 +-
 6 files changed, 101 insertions(+), 21 deletions(-)
 create mode 100644 src/main/java/org/kohsuke/github/GHLabel.java

diff --git a/src/main/java/org/kohsuke/github/GHIssue.java b/src/main/java/org/kohsuke/github/GHIssue.java
index 53db997adc..4978427c69 100644
--- a/src/main/java/org/kohsuke/github/GHIssue.java
+++ b/src/main/java/org/kohsuke/github/GHIssue.java
@@ -52,29 +52,17 @@ public class GHIssue extends GHObject {
     protected String closed_at;
     protected int comments;
     protected String body;
-    protected List<Label> labels;
+    protected List<GHLabel> labels;
     protected GHUser user;
     protected String title, html_url;
     protected GHIssue.PullRequest pull_request;
     protected GHMilestone milestone;
     protected GHUser closed_by;
 
-    public static class Label {
-        private String url;
-        private String name;
-        private String color;
-		
-        public String getUrl() {
-			return url;
-		}
-		
-        public String getName() {
-			return name;
-		}
-		
-        public String getColor() {
-			return color;
-		}
+    /**
+     * @deprecated use {@link GHLabel}
+     */
+    public static class Label extends GHLabel {
     }
     
     /*package*/ GHIssue wrap(GHRepository owner) {
@@ -134,9 +122,9 @@ public GHIssueState getState() {
         return Enum.valueOf(GHIssueState.class, state.toUpperCase(Locale.ENGLISH));
     }
 
-    public Collection<Label> getLabels() throws IOException {
+    public Collection<GHLabel> getLabels() throws IOException {
         if(labels == null){
-            return Collections.EMPTY_LIST;
+            return Collections.emptyList();
         }
         return Collections.unmodifiableList(labels);
     }
diff --git a/src/main/java/org/kohsuke/github/GHLabel.java b/src/main/java/org/kohsuke/github/GHLabel.java
new file mode 100644
index 0000000000..5d1b3b48a2
--- /dev/null
+++ b/src/main/java/org/kohsuke/github/GHLabel.java
@@ -0,0 +1,37 @@
+package org.kohsuke.github;
+
+import java.io.IOException;
+
+/**
+ * @author Kohsuke Kawaguchi
+ * @see GHIssue#getLabels()
+ * @see GHRepository#listLabels()
+ */
+public class GHLabel {
+    private String url, name, color;
+    private GHRepository repo;
+
+    public String getUrl() {
+        return url;
+    }
+
+    public String getName() {
+        return name;
+    }
+
+    /**
+     * Color code without leading '#', such as 'f29513'
+     */
+    public String getColor() {
+        return color;
+    }
+
+    /*package*/ GHLabel wrapUp(GHRepository repo) {
+        this.repo = repo;
+        return this;
+    }
+
+    public void delete() throws IOException {
+        repo.root.retrieve().method("DELETE").to(url);
+    }
+}
diff --git a/src/main/java/org/kohsuke/github/GHPullRequest.java b/src/main/java/org/kohsuke/github/GHPullRequest.java
index b961a83654..828b93b180 100644
--- a/src/main/java/org/kohsuke/github/GHPullRequest.java
+++ b/src/main/java/org/kohsuke/github/GHPullRequest.java
@@ -127,7 +127,7 @@ public Date getMergedAt() {
     }
 
 	@Override
-	public Collection<Label> getLabels() throws IOException {
+	public Collection<GHLabel> getLabels() throws IOException {
         fetchIssue();
 		return super.getLabels();
 	}
diff --git a/src/main/java/org/kohsuke/github/GHRepository.java b/src/main/java/org/kohsuke/github/GHRepository.java
index c0f36153c2..7d75222642 100644
--- a/src/main/java/org/kohsuke/github/GHRepository.java
+++ b/src/main/java/org/kohsuke/github/GHRepository.java
@@ -760,6 +760,36 @@ protected void wrapUp(GHEventInfo[] page) {
         };
     }
 
+    /**
+     * Lists labels in this repository.
+     *
+     * https://developer.github.com/v3/issues/labels/#list-all-labels-for-this-repository
+     */
+    public PagedIterable<GHLabel> listLabels() throws IOException {
+        return new PagedIterable<GHLabel>() {
+            public PagedIterator<GHLabel> iterator() {
+                return new PagedIterator<GHLabel>(root.retrieve().asIterator(getApiTailUrl("labels"), GHLabel[].class)) {
+                    @Override
+                    protected void wrapUp(GHLabel[] page) {
+                        for (GHLabel c : page)
+                            c.wrapUp(GHRepository.this);
+                    }
+                };
+            }
+        };
+    }
+
+    public GHLabel getLabel(String name) throws IOException {
+        return root.retrieve().to(getApiTailUrl("labels/"+name), GHLabel.class).wrapUp(this);
+    }
+
+    public GHLabel createLabel(String name, String color) throws IOException {
+        return root.retrieve().method("POST")
+                .with("name",name)
+                .with("color",color)
+                .to(getApiTailUrl("labels"), GHLabel.class).wrapUp(this);
+    }
+
     /**
      * 
      * See https://api.github.com/hooks for possible names and their configuration scheme.
diff --git a/src/test/java/org/kohsuke/github/AppTest.java b/src/test/java/org/kohsuke/github/AppTest.java
index 3afb464618..f91d10a1e6 100755
--- a/src/test/java/org/kohsuke/github/AppTest.java
+++ b/src/test/java/org/kohsuke/github/AppTest.java
@@ -12,6 +12,7 @@
 import java.net.URL;
 import java.util.*;
 import java.util.Map.Entry;
+import java.util.regex.Pattern;
 
 /**
  * Unit test for simple App.
@@ -665,6 +666,30 @@ public void testReadme() throws IOException {
         assertEquals(readme.getContent(),"This is a markdown readme.\n");
     }
 
+    @Test
+    public void testRepoLabel() throws IOException {
+        GHRepository r = gitHub.getRepository("github-api-test-org/test-labels");
+        List<GHLabel> lst = r.listLabels().asList();
+        for (GHLabel l : lst) {
+            System.out.println(l.getName());
+        }
+        assertTrue(lst.size() > 5);
+        GHLabel e = r.getLabel("enhancement");
+        assertEquals("enhancement",e.getName());
+        assertNotNull(e.getUrl());
+        assertTrue(Pattern.matches("[0-9a-fA-F]{6}",e.getColor()));
+
+        {// CRUD
+            GHLabel t = r.createLabel("test", "123456");
+            GHLabel t2 = r.getLabel("test");
+            assertEquals(t.getName(), t2.getName());
+            assertEquals(t.getColor(), "123456");
+            assertEquals(t.getColor(), t2.getColor());
+            assertEquals(t.getUrl(), t2.getUrl());
+            t.delete();
+        }
+    }
+
     private void kohsuke() {
         String login = getUser().getLogin();
         Assume.assumeTrue(login.equals("kohsuke") || login.equals("kohsuke2"));
diff --git a/src/test/java/org/kohsuke/github/PullRequestTest.java b/src/test/java/org/kohsuke/github/PullRequestTest.java
index 174d38b2c0..de3b895419 100644
--- a/src/test/java/org/kohsuke/github/PullRequestTest.java
+++ b/src/test/java/org/kohsuke/github/PullRequestTest.java
@@ -24,7 +24,7 @@ public void setLabels() throws Exception {
         String label = rnd.next();
         p.setLabels(label);
 
-        Collection<GHIssue.Label> labels = getRepository().getPullRequest(p.getNumber()).getLabels();
+        Collection<GHLabel> labels = getRepository().getPullRequest(p.getNumber()).getLabels();
         assertEquals(1, labels.size());
         assertEquals(label, labels.iterator().next().getName());
     }

From 7bbe0f7e8a0222a123b397d915db71a724f36087 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sun, 15 Feb 2015 07:02:50 -0800
Subject: [PATCH 04/19] Allow the client to explicitly control proxy

Fixes issue #109.
---
 .../java/org/kohsuke/github/GitHubBuilder.java   | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/src/main/java/org/kohsuke/github/GitHubBuilder.java b/src/main/java/org/kohsuke/github/GitHubBuilder.java
index 0aa9532506..e794fb14ca 100644
--- a/src/main/java/org/kohsuke/github/GitHubBuilder.java
+++ b/src/main/java/org/kohsuke/github/GitHubBuilder.java
@@ -6,6 +6,9 @@
 import java.io.FileInputStream;
 import java.io.FileNotFoundException;
 import java.io.IOException;
+import java.net.HttpURLConnection;
+import java.net.Proxy;
+import java.net.URL;
 import java.util.Map;
 import java.util.Properties;
 
@@ -153,6 +156,19 @@ public GitHubBuilder withConnector(HttpConnector connector) {
         return this;
     }
 
+    /**
+     * Configures {@linkplain #withConnector(HttpConnector) connector}
+     * that uses HTTP library in JRE but use a specific proxy, instead of
+     * the system default one.
+     */
+    public GitHubBuilder withProxy(final Proxy p) {
+        return withConnector(new HttpConnector() {
+            public HttpURLConnection connect(URL url) throws IOException {
+                return (HttpURLConnection) url.openConnection(p);
+            }
+        });
+    }
+
     public GitHub build() throws IOException {
         return new GitHub(endpoint, user, oauthToken, password, connector);
     }

From 9afad71b0f6ce188109cc3decad57a054a23740e Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sun, 15 Feb 2015 07:13:33 -0800
Subject: [PATCH 05/19] Newly created user object resets root to null.

Fixes issue #111.
Test case from KostyaSha
---
 .../java/org/kohsuke/github/GHPullRequest.java  |  2 +-
 src/test/java/Foo.java                          | 17 ++++++-----------
 .../org/kohsuke/github/PullRequestTest.java     | 16 ++++++++++++++++
 3 files changed, 23 insertions(+), 12 deletions(-)

diff --git a/src/main/java/org/kohsuke/github/GHPullRequest.java b/src/main/java/org/kohsuke/github/GHPullRequest.java
index 828b93b180..7510cdf6c5 100644
--- a/src/main/java/org/kohsuke/github/GHPullRequest.java
+++ b/src/main/java/org/kohsuke/github/GHPullRequest.java
@@ -194,7 +194,7 @@ public int getChangedFiles() throws IOException {
     private void populate() throws IOException {
         if (merged_by!=null)    return; // already populated
 
-        root.retrieve().to(url, this);
+        root.retrieve().to(url, this).wrapUp(owner);
     }
 
     /**
diff --git a/src/test/java/Foo.java b/src/test/java/Foo.java
index cccfa23115..5df13995c5 100644
--- a/src/test/java/Foo.java
+++ b/src/test/java/Foo.java
@@ -1,22 +1,17 @@
-import org.kohsuke.github.GHIssue;
-import org.kohsuke.github.GHOrganization;
-import org.kohsuke.github.GHTeam;
+import org.kohsuke.github.GHRepository;
 import org.kohsuke.github.GitHub;
-import org.kohsuke.github.PagedIterable;
-import org.kohsuke.github.PagedSearchIterable;
 
-import java.util.Arrays;
-import java.util.Map;
+import java.util.Collection;
 
 /**
  * @author Kohsuke Kawaguchi
  */
 public class Foo {
     public static void main(String[] args) throws Exception {
-        PagedSearchIterable<GHIssue> reviewbybees = GitHub.connect().searchIssues().mentions("reviewbybees").isOpen().list();
-        for (GHIssue r : reviewbybees) {
-            System.out.println(r.getTitle());
+        Collection<GHRepository> lst = GitHub.connect().getUser("kohsuke").getRepositories().values();
+        for (GHRepository r : lst) {
+            System.out.println(r.getName());
         }
-        System.out.println("total="+reviewbybees.getTotalCount());
+        System.out.println(lst.size());
     }
 }
diff --git a/src/test/java/org/kohsuke/github/PullRequestTest.java b/src/test/java/org/kohsuke/github/PullRequestTest.java
index de3b895419..c513860eae 100644
--- a/src/test/java/org/kohsuke/github/PullRequestTest.java
+++ b/src/test/java/org/kohsuke/github/PullRequestTest.java
@@ -38,6 +38,22 @@ public void setAssignee() throws Exception {
         assertEquals(user, getRepository().getPullRequest(p.getNumber()).getAssignee());
     }
 
+    @Test
+    public void testGetUser() throws IOException {
+        GHPullRequest p = getRepository().createPullRequest(rnd.next(), "stable", "master", "## test");
+        GHPullRequest prSingle = getRepository().getPullRequest(p.getNumber());
+        assertNotNull(prSingle.getUser().root);
+        prSingle.getMergeable();
+        assertNotNull(prSingle.getUser().root);
+
+        PagedIterable<GHPullRequest> ghPullRequests = getRepository().listPullRequests(GHIssueState.OPEN);
+        for (GHPullRequest pr : ghPullRequests) {
+            assertNotNull(pr.getUser().root);
+            assertFalse(pr.getMergeable());
+            assertNotNull(pr.getUser().root);
+        }
+    }
+
     @After
     public void cleanUp() throws Exception {
         for (GHPullRequest pr : getRepository().getPullRequests(GHIssueState.OPEN)) {

From e2e23293015d69e2430218c6282826f3c6b1365b Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sun, 15 Feb 2015 07:16:18 -0800
Subject: [PATCH 06/19] Noting issue #60 that this method can return null

---
 src/main/java/org/kohsuke/github/GHIssue.java | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/src/main/java/org/kohsuke/github/GHIssue.java b/src/main/java/org/kohsuke/github/GHIssue.java
index 4978427c69..738b50a75c 100644
--- a/src/main/java/org/kohsuke/github/GHIssue.java
+++ b/src/main/java/org/kohsuke/github/GHIssue.java
@@ -225,7 +225,15 @@ public GHUser getAssignee() {
 	public GHUser getUser() {
         return user;
 	}
-	
+
+    /**
+     * Reports who has closed the issue.
+     *
+     * <p>
+     * Note that GitHub doesn't always seem to report this information
+     * even for an issue that's already closed. See
+     * https://github.com/kohsuke/github-api/issues/60.
+     */
 	public GHUser getClosedBy() {
 		if(!"closed".equals(state)) return null;
 		if(closed_by != null) return closed_by;

From 541dac1aee1be51e4bb8e6f64e0bab0ec4e100f4 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sun, 15 Feb 2015 07:34:15 -0800
Subject: [PATCH 07/19] Use getApiTailUrl for consistency

---
 .../java/org/kohsuke/github/GHRepository.java | 43 ++++++++++---------
 1 file changed, 22 insertions(+), 21 deletions(-)

diff --git a/src/main/java/org/kohsuke/github/GHRepository.java b/src/main/java/org/kohsuke/github/GHRepository.java
index 7d75222642..a4bf4ae78d 100644
--- a/src/main/java/org/kohsuke/github/GHRepository.java
+++ b/src/main/java/org/kohsuke/github/GHRepository.java
@@ -195,7 +195,7 @@ public GHUser getOwner() throws IOException {
     }
 
     public GHIssue getIssue(int id) throws IOException {
-        return root.retrieve().to("/repos/" + owner.login + "/" + name + "/issues/" + id, GHIssue.class).wrap(this);
+        return root.retrieve().to(getApiTailUrl("issues/" + id), GHIssue.class).wrap(this);
     }
 
     public GHIssueBuilder createIssue(String title) {
@@ -208,8 +208,8 @@ public List<GHIssue> getIssues(GHIssueState state) throws IOException {
 
     public List<GHIssue> getIssues(GHIssueState state, GHMilestone milestone) throws IOException {
         return Arrays.asList(GHIssue.wrap(root.retrieve()
-                .to(String.format("/repos/%s/%s/issues?state=%s&milestone=%s", owner.login, name,
-                        state.toString().toLowerCase(), milestone == null ? "none" : "" + milestone.getNumber()),
+                .to(getApiTailUrl(String.format("issues?state=%s&milestone=%s",
+                        state.toString().toLowerCase(), milestone == null ? "none" : "" + milestone.getNumber())),
                         GHIssue[].class
                 ), this));
     }
@@ -371,7 +371,7 @@ public PagedIterable<GHUser> listCollaborators() throws IOException {
         return new PagedIterable<GHUser>() {
             public PagedIterator<GHUser> iterator() {
 
-                return new PagedIterator<GHUser>(root.retrieve().asIterator("/repos/" + owner.login + "/" + name + "/collaborators", GHUser[].class)) {
+                return new PagedIterator<GHUser>(root.retrieve().asIterator(getApiTailUrl("collaborators"), GHUser[].class)) {
 
                     @Override
                     protected void wrapUp(GHUser[] users) {
@@ -392,7 +392,7 @@ protected void wrapUp(GHUser[] users) {
      */
     public Set<String> getCollaboratorNames() throws IOException {
         Set<String> r = new HashSet<String>();
-        for (GHUser u : GHUser.wrap(root.retrieve().to("/repos/" + owner.login + "/" + name + "/collaborators", GHUser[].class),root))
+        for (GHUser u : GHUser.wrap(root.retrieve().to(getApiTailUrl("collaborators"), GHUser[].class),root))
             r.add(u.login);
         return r;
     }
@@ -401,7 +401,7 @@ public Set<String> getCollaboratorNames() throws IOException {
      * If this repository belongs to an organization, return a set of teams.
      */
     public Set<GHTeam> getTeams() throws IOException {
-        return Collections.unmodifiableSet(new HashSet<GHTeam>(Arrays.asList(GHTeam.wrapUp(root.retrieve().to("/repos/" + owner.login + "/" + name + "/teams", GHTeam[].class), root.getOrganization(owner.login)))));
+        return Collections.unmodifiableSet(new HashSet<GHTeam>(Arrays.asList(GHTeam.wrapUp(root.retrieve().to(getApiTailUrl("teams"), GHTeam[].class), root.getOrganization(owner.login)))));
     }
 
     public void addCollaborators(GHUser... users) throws IOException {
@@ -423,7 +423,7 @@ public void removeCollaborators(Collection<GHUser> users) throws IOException {
     private void modifyCollaborators(Collection<GHUser> users, String method) throws IOException {
         verifyMine();
         for (GHUser user : users) {
-            new Requester(root).method(method).to("/repos/" + owner.login + "/" + name + "/collaborators/" + user.getLogin());
+            new Requester(root).method(method).to(getApiTailUrl("collaborators/" + user.getLogin()));
         }
     }
 
@@ -431,14 +431,14 @@ public void setEmailServiceHook(String address) throws IOException {
         Map<String, String> config = new HashMap<String, String>();
         config.put("address", address);
         new Requester(root).method("POST").with("name", "email").with("config", config).with("active", "true")
-                .to(String.format("/repos/%s/%s/hooks", owner.login, name));
+                .to(getApiTailUrl("hooks"));
     }
 
     private void edit(String key, String value) throws IOException {
         Requester requester = new Requester(root);
         if (!key.equals("name"))
             requester.with("name", name);   // even when we don't change the name, we need to send it in
-        requester.with(key, value).method("PATCH").to("/repos/" + owner.login + "/" + name);
+        requester.with(key, value).method("PATCH").to(getApiTailUrl(""));
     }
 
     /**
@@ -479,7 +479,7 @@ public void setHomepage(String value) throws IOException {
      */
     public void delete() throws IOException {
         try {
-            new Requester(root).method("DELETE").to("/repos/" + owner.login + "/" + name);
+            new Requester(root).method("DELETE").to(getApiTailUrl(""));
         } catch (FileNotFoundException x) {
             throw (FileNotFoundException) new FileNotFoundException("Failed to delete " + owner.login + "/" + name + "; might not exist, or you might need the delete_repo scope in your token: http://stackoverflow.com/a/19327004/12916").initCause(x);
         }
@@ -492,7 +492,7 @@ public void delete() throws IOException {
      *      Newly forked repository that belong to you.
      */
     public GHRepository fork() throws IOException {
-        return new Requester(root).method("POST").to("/repos/" + owner.login + "/" + name + "/forks", GHRepository.class).wrap(root);
+        return new Requester(root).method("POST").to(getApiTailUrl("forks"), GHRepository.class).wrap(root);
     }
 
     /**
@@ -502,7 +502,7 @@ public GHRepository fork() throws IOException {
      *      Newly forked repository that belong to you.
      */
     public GHRepository forkTo(GHOrganization org) throws IOException {
-        new Requester(root).to(String.format("/repos/%s/%s/forks?org=%s", owner.login, name, org.getLogin()));
+        new Requester(root).to(getApiTailUrl("forks?org="+org.getLogin()));
 
         // this API is asynchronous. we need to wait for a bit
         for (int i=0; i<10; i++) {
@@ -521,7 +521,7 @@ public GHRepository forkTo(GHOrganization org) throws IOException {
      * Retrieves a specified pull request.
      */
     public GHPullRequest getPullRequest(int i) throws IOException {
-        return root.retrieve().to("/repos/" + owner.login + '/' + name + "/pulls/" + i, GHPullRequest.class).wrapUp(this);
+        return root.retrieve().to(getApiTailUrl("pulls/" + i), GHPullRequest.class).wrapUp(this);
     }
 
     /**
@@ -539,7 +539,7 @@ public List<GHPullRequest> getPullRequests(GHIssueState state) throws IOExceptio
     public PagedIterable<GHPullRequest> listPullRequests(final GHIssueState state) {
         return new PagedIterable<GHPullRequest>() {
             public PagedIterator<GHPullRequest> iterator() {
-                return new PagedIterator<GHPullRequest>(root.retrieve().asIterator(String.format("/repos/%s/%s/pulls?state=%s", owner.login, name, state.name().toLowerCase(Locale.ENGLISH)), GHPullRequest[].class)) {
+                return new PagedIterator<GHPullRequest>(root.retrieve().asIterator(getApiTailUrl("pulls?state="+state.name().toLowerCase(Locale.ENGLISH)), GHPullRequest[].class)) {
                     @Override
                     protected void wrapUp(GHPullRequest[] page) {
                         for (GHPullRequest pr : page)
@@ -578,14 +578,14 @@ public GHPullRequest createPullRequest(String title, String head, String base, S
      */
     public List<GHHook> getHooks() throws IOException {
         List<GHHook> list = new ArrayList<GHHook>(Arrays.asList(
-                root.retrieve().to(String.format("/repos/%s/%s/hooks", owner.login, name), GHHook[].class)));
+                root.retrieve().to(getApiTailUrl("hooks"), GHHook[].class)));
         for (GHHook h : list)
             h.wrap(this);
         return list;
     }
 
     public GHHook getHook(int id) throws IOException {
-        return root.retrieve().to(String.format("/repos/%s/%s/hooks/%d", owner.login, name, id), GHHook.class).wrap(this);
+        return root.retrieve().to(getApiTailUrl("hooks/"+id), GHHook.class).wrap(this);
     }
 
     /**
@@ -597,7 +597,7 @@ public GHHook getHook(int id) throws IOException {
      * @throws IOException on failure communicating with GitHub
      */
     public GHCompare getCompare(String id1, String id2) throws IOException {
-        GHCompare compare = root.retrieve().to(String.format("/repos/%s/%s/compare/%s...%s", owner.login, name, id1, id2), GHCompare.class);
+        GHCompare compare = root.retrieve().to(getApiTailUrl(String.format("compare/%s...%s", id1, id2)), GHCompare.class);
         return compare.wrap(this);
     }
 
@@ -968,7 +968,7 @@ public GHContent getFileContent(String path) throws IOException {
 
     public GHContent getFileContent(String path, String ref) throws IOException {
         Requester requester = root.retrieve();
-        String target = String.format("/repos/%s/%s/contents/%s", owner.login, name, path);
+        String target = getApiTailUrl("contents/"+path);
 
         if (ref != null)
             target = target + "?ref=" + ref;
@@ -982,7 +982,7 @@ public List<GHContent> getDirectoryContent(String path) throws IOException {
 
     public List<GHContent> getDirectoryContent(String path, String ref) throws IOException {
         Requester requester = root.retrieve();
-        String target = String.format("/repos/%s/%s/contents/%s", owner.login, name, path);
+        String target = getApiTailUrl("contents/"+path);
 
         if (ref != null)
             target = target + "?ref=" + ref;
@@ -1046,7 +1046,7 @@ public GHDeployKey addDeployKey(String title,String key) throws IOException {
 	
 	public List<GHDeployKey> getDeployKeys() throws IOException{
 		 List<GHDeployKey> list = new ArrayList<GHDeployKey>(Arrays.asList(
-	                root.retrieve().to(String.format("/repos/%s/%s/keys", owner.login, name), GHDeployKey[].class)));
+	                root.retrieve().to(getApiTailUrl("keys"), GHDeployKey[].class)));
 	        for (GHDeployKey h : list)
 	            h.wrap(this);
 	        return list;	
@@ -1075,6 +1075,7 @@ public boolean equals(Object obj) {
     }
 
     String getApiTailUrl(String tail) {
-        return "/repos/" + owner.login + "/" + name +'/'+tail;
+        if (tail.length()>0 && !tail.startsWith("/"))    tail='/'+tail;
+        return "/repos/" + owner.login + "/" + name +tail;
     }
 }

From 762a32eb6dec4965830c49828188a96149b63b10 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sun, 15 Feb 2015 07:45:11 -0800
Subject: [PATCH 08/19] Added repository watch listing

---
 .../java/org/kohsuke/github/GHRepository.java | 24 ++++++++++++++++---
 src/main/java/org/kohsuke/github/GHUser.java  | 23 ++++++++++++++++++
 src/test/java/org/kohsuke/github/AppTest.java | 19 +++++++++++++++
 .../org/kohsuke/github/RepositoryTest.java    |  2 --
 4 files changed, 63 insertions(+), 5 deletions(-)

diff --git a/src/main/java/org/kohsuke/github/GHRepository.java b/src/main/java/org/kohsuke/github/GHRepository.java
index a4bf4ae78d..64cad27508 100644
--- a/src/main/java/org/kohsuke/github/GHRepository.java
+++ b/src/main/java/org/kohsuke/github/GHRepository.java
@@ -585,7 +585,7 @@ public List<GHHook> getHooks() throws IOException {
     }
 
     public GHHook getHook(int id) throws IOException {
-        return root.retrieve().to(getApiTailUrl("hooks/"+id), GHHook.class).wrap(this);
+        return root.retrieve().to(getApiTailUrl("hooks/" + id), GHHook.class).wrap(this);
     }
 
     /**
@@ -790,6 +790,24 @@ public GHLabel createLabel(String name, String color) throws IOException {
                 .to(getApiTailUrl("labels"), GHLabel.class).wrapUp(this);
     }
 
+    /**
+     * Lists all the subscribers (aka watchers.)
+     *
+     * https://developer.github.com/v3/activity/watching/
+     */
+    public PagedIterable<GHUser> listSubscribers() {
+        return new PagedIterable<GHUser>() {
+            public PagedIterator<GHUser> iterator() {
+                return new PagedIterator<GHUser>(root.retrieve().asIterator(getApiTailUrl("subscribers"), GHUser[].class)) {
+                    protected void wrapUp(GHUser[] page) {
+                        for (GHUser c : page)
+                            c.wrapUp(root);
+                    }
+                };
+            }
+        };
+    }
+
     /**
      * 
      * See https://api.github.com/hooks for possible names and their configuration scheme.
@@ -968,7 +986,7 @@ public GHContent getFileContent(String path) throws IOException {
 
     public GHContent getFileContent(String path, String ref) throws IOException {
         Requester requester = root.retrieve();
-        String target = getApiTailUrl("contents/"+path);
+        String target = getApiTailUrl("contents/" + path);
 
         if (ref != null)
             target = target + "?ref=" + ref;
@@ -982,7 +1000,7 @@ public List<GHContent> getDirectoryContent(String path) throws IOException {
 
     public List<GHContent> getDirectoryContent(String path, String ref) throws IOException {
         Requester requester = root.retrieve();
-        String target = getApiTailUrl("contents/"+path);
+        String target = getApiTailUrl("contents/" + path);
 
         if (ref != null)
             target = target + "?ref=" + ref;
diff --git a/src/main/java/org/kohsuke/github/GHUser.java b/src/main/java/org/kohsuke/github/GHUser.java
index 7ebd356d9b..2242f4a6e4 100644
--- a/src/main/java/org/kohsuke/github/GHUser.java
+++ b/src/main/java/org/kohsuke/github/GHUser.java
@@ -69,6 +69,24 @@ public GHPersonSet<GHUser> getFollowers() throws IOException {
         return new GHPersonSet<GHUser>(Arrays.asList(wrap(followers,root)));
     }
 
+    /**
+     * Lists all the subscribed (aka watched) repositories.
+     *
+     * https://developer.github.com/v3/activity/watching/
+     */
+    public PagedIterable<GHRepository> listSubscriptions() {
+        return new PagedIterable<GHRepository>() {
+            public PagedIterator<GHRepository> iterator() {
+                return new PagedIterator<GHRepository>(root.retrieve().asIterator(getApiTailUrl("subscriptions"), GHRepository[].class)) {
+                    protected void wrapUp(GHRepository[] page) {
+                        for (GHRepository c : page)
+                            c.wrap(root);
+                    }
+                };
+            }
+        };
+    }
+
     /**
      * Returns true if this user belongs to the specified organization.
      */
@@ -162,4 +180,9 @@ public boolean equals(Object obj) {
         }
         return false;
     }
+
+    String getApiTailUrl(String tail) {
+        if (tail.length()>0 && !tail.startsWith("/"))    tail='/'+tail;
+        return "/users/" + login + tail;
+    }
 }
diff --git a/src/test/java/org/kohsuke/github/AppTest.java b/src/test/java/org/kohsuke/github/AppTest.java
index f91d10a1e6..a13f5423f5 100755
--- a/src/test/java/org/kohsuke/github/AppTest.java
+++ b/src/test/java/org/kohsuke/github/AppTest.java
@@ -690,6 +690,25 @@ public void testRepoLabel() throws IOException {
         }
     }
 
+    @Test
+    public void testSubscribers() throws IOException {
+        boolean kohsuke = false;
+        GHRepository mr = gitHub.getRepository("kohsuke/github-api");
+        for (GHUser u : mr.listSubscribers()) {
+            System.out.println(u.getLogin());
+            kohsuke |= u.getLogin().equals("kohsuke");
+        }
+        assertTrue(kohsuke);
+        System.out.println("---");
+
+        boolean githubApi = false;
+        for (GHRepository r : gitHub.getUser("kohsuke").listRepositories()) {
+            System.out.println(r.getName());
+            githubApi |= r.equals(mr);
+        }
+        assertTrue(githubApi);
+    }
+
     private void kohsuke() {
         String login = getUser().getLogin();
         Assume.assumeTrue(login.equals("kohsuke") || login.equals("kohsuke2"));
diff --git a/src/test/java/org/kohsuke/github/RepositoryTest.java b/src/test/java/org/kohsuke/github/RepositoryTest.java
index 1bc612b29c..d7d9626032 100644
--- a/src/test/java/org/kohsuke/github/RepositoryTest.java
+++ b/src/test/java/org/kohsuke/github/RepositoryTest.java
@@ -85,7 +85,5 @@ protected void wrapUp(GHUser[] page) {
         Assert.assertTrue(returnIterator2.hasNext());
         user = returnIterator1.next();
         Assert.assertEquals(user, user2);
-
-
     }
 }

From c965b9cc24f08daa81d2965d8f92be0ea23da6d1 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sun, 15 Feb 2015 08:20:28 -0800
Subject: [PATCH 09/19] follow up fix to the GHRepository.getApiTailUrl()
 change

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

diff --git a/src/main/java/org/kohsuke/github/GHRelease.java b/src/main/java/org/kohsuke/github/GHRelease.java
index 0a963b1887..88d978258e 100644
--- a/src/main/java/org/kohsuke/github/GHRelease.java
+++ b/src/main/java/org/kohsuke/github/GHRelease.java
@@ -120,7 +120,7 @@ static GHRelease[] wrap(GHRelease[] releases, GHRepository owner) {
     public GHAsset uploadAsset(File file, String contentType) throws IOException {
         Requester builder = new Requester(owner.root);
 
-        String url = format("https://uploads.github.com%sreleases/%d/assets?name=%s",
+        String url = format("https://uploads.github.com%s/releases/%d/assets?name=%s",
                 owner.getApiTailUrl(""), getId(), file.getName());
         return builder.contentType(contentType)
                 .with(new FileInputStream(file))

From 60dce94a47d517676142e9cbfe1db81fc6da4ee4 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sun, 15 Feb 2015 08:25:49 -0800
Subject: [PATCH 10/19] renamed to created RepositoryTest

---
 .../github/{RepositoryTest.java => RepositoryMockTest.java}     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename src/test/java/org/kohsuke/github/{RepositoryTest.java => RepositoryMockTest.java} (98%)

diff --git a/src/test/java/org/kohsuke/github/RepositoryTest.java b/src/test/java/org/kohsuke/github/RepositoryMockTest.java
similarity index 98%
rename from src/test/java/org/kohsuke/github/RepositoryTest.java
rename to src/test/java/org/kohsuke/github/RepositoryMockTest.java
index d7d9626032..8c12ed640a 100644
--- a/src/test/java/org/kohsuke/github/RepositoryTest.java
+++ b/src/test/java/org/kohsuke/github/RepositoryMockTest.java
@@ -14,7 +14,7 @@
 /**
  * @author Luciano P. Sabenca (luciano.sabenca [at] movile [com] | lucianosabenca [at] gmail [dot] com
  */
-public class RepositoryTest {
+public class RepositoryMockTest {
 
     @Mock
     GitHub mockGitHub;

From a8cf4a71200c63acfd4e8c996f882e8cecfb169a Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sun, 15 Feb 2015 08:31:57 -0800
Subject: [PATCH 11/19] Added watch API support.

Fixes issue #130
---
 .../java/org/kohsuke/github/GHRepository.java | 24 +++++++
 .../org/kohsuke/github/GHSubscription.java    | 62 +++++++++++++++++++
 .../org/kohsuke/github/RepositoryTest.java    | 27 ++++++++
 3 files changed, 113 insertions(+)
 create mode 100644 src/main/java/org/kohsuke/github/GHSubscription.java
 create mode 100644 src/test/java/org/kohsuke/github/RepositoryTest.java

diff --git a/src/main/java/org/kohsuke/github/GHRepository.java b/src/main/java/org/kohsuke/github/GHRepository.java
index 64cad27508..c1d29ccbdd 100644
--- a/src/main/java/org/kohsuke/github/GHRepository.java
+++ b/src/main/java/org/kohsuke/github/GHRepository.java
@@ -1069,6 +1069,30 @@ public List<GHDeployKey> getDeployKeys() throws IOException{
 	            h.wrap(this);
 	        return list;	
 	}
+
+    /**
+     * Subscribes to this repository to get notifications.
+     */
+    public GHSubscription subscribe(boolean subscribed, boolean ignored) throws IOException {
+        return new Requester(root)
+            .with("subscribed", subscribed)
+            .with("ignored", ignored)
+            .method("PUT").to(getApiTailUrl("subscription"), GHSubscription.class).wrapUp(this);
+    }
+
+    /**
+     * Returns the current subscription.
+     *
+     * @return null if no subscription exists.
+     */
+    public GHSubscription getSubscription() throws IOException {
+        try {
+            return new Requester(root).to(getApiTailUrl("subscription"), GHSubscription.class).wrapUp(this);
+        } catch (FileNotFoundException e) {
+            return null;
+        }
+    }
+
 	
 	
 
diff --git a/src/main/java/org/kohsuke/github/GHSubscription.java b/src/main/java/org/kohsuke/github/GHSubscription.java
new file mode 100644
index 0000000000..bc5dcb92e4
--- /dev/null
+++ b/src/main/java/org/kohsuke/github/GHSubscription.java
@@ -0,0 +1,62 @@
+package org.kohsuke.github;
+
+import java.io.IOException;
+import java.util.Date;
+
+/**
+ * Represents your subscribing to a repository.
+ *
+ * @author Kohsuke Kawaguchi
+ */
+public class GHSubscription {
+    private String created_at, url, repository_url, reason;
+    private boolean subscribed, ignored;
+
+    private GitHub root;
+    private GHRepository repo;
+
+    public Date getCreatedAt() {
+        return GitHub.parseDate(created_at);
+    }
+
+    public String getUrl() {
+        return url;
+    }
+
+    public String getRepositoryUrl() {
+        return repository_url;
+    }
+
+    public String getReason() {
+        return reason;
+    }
+
+    public boolean isSubscribed() {
+        return subscribed;
+    }
+
+    public boolean isIgnored() {
+        return ignored;
+    }
+
+    public GHRepository getRepository() {
+        return repo;
+    }
+
+    /**
+     * Removes this subscription.
+     */
+    public void delete() throws IOException {
+        new Requester(root).method("DELETE").to(url);
+    }
+
+    GHSubscription wrapUp(GHRepository repo) {
+        this.repo = repo;
+        return wrapUp(repo.root);
+    }
+
+    GHSubscription wrapUp(GitHub root) {
+        this.root = root;
+        return this;
+    }
+}
diff --git a/src/test/java/org/kohsuke/github/RepositoryTest.java b/src/test/java/org/kohsuke/github/RepositoryTest.java
new file mode 100644
index 0000000000..2a5b62baab
--- /dev/null
+++ b/src/test/java/org/kohsuke/github/RepositoryTest.java
@@ -0,0 +1,27 @@
+package org.kohsuke.github;
+
+import org.junit.Test;
+
+import java.io.IOException;
+
+/**
+ * @author Kohsuke Kawaguchi
+ */
+public class RepositoryTest extends AbstractGitHubApiTestBase {
+    @Test
+    public void subscription() throws Exception {
+        GHRepository r = getRepository();
+        assertNull(r.getSubscription());
+
+        GHSubscription s = r.subscribe(true, false);
+        assertEquals(s.getRepository(), r);
+
+        s.delete();
+
+        assertNull(r.getSubscription());
+    }
+
+    private GHRepository getRepository() throws IOException {
+        return gitHub.getOrganization("github-api-test-org").getRepository("jenkins");
+    }
+}

From 29f826448a9e073153614d9907a7180b79d398b6 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sun, 15 Feb 2015 08:35:31 -0800
Subject: [PATCH 12/19] Added a convenience method.

See: issue #134
---
 src/main/java/org/kohsuke/github/GHIssue.java | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/src/main/java/org/kohsuke/github/GHIssue.java b/src/main/java/org/kohsuke/github/GHIssue.java
index 738b50a75c..0a33cdc013 100644
--- a/src/main/java/org/kohsuke/github/GHIssue.java
+++ b/src/main/java/org/kohsuke/github/GHIssue.java
@@ -246,10 +246,17 @@ public int getCommentsCount(){
 		return comments;
 	}
 
+    /**
+     * Returns non-null if this issue is a shadow of a pull request.
+     */
 	public PullRequest getPullRequest() {
 		return pull_request;
 	}
 
+    public boolean isPullRequest() {
+        return pull_request!=null;
+    }
+
 	public GHMilestone getMilestone() {
 		return milestone;
 	}

From 11fcb9d4561dee82d6310b15273e2b7307a5f3db Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sun, 15 Feb 2015 08:42:32 -0800
Subject: [PATCH 13/19] Report the repository the push happened to

Fixes issue #144.
---
 src/main/java/org/kohsuke/github/GHEventPayload.java | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/src/main/java/org/kohsuke/github/GHEventPayload.java b/src/main/java/org/kohsuke/github/GHEventPayload.java
index 704d7f8e81..4e718a2dc6 100644
--- a/src/main/java/org/kohsuke/github/GHEventPayload.java
+++ b/src/main/java/org/kohsuke/github/GHEventPayload.java
@@ -120,6 +120,7 @@ public static class Push extends GHEventPayload {
         private String ref;
         private int size;
         private List<PushCommit> commits;
+        private GHRepository repository;
 
         /**
          * The SHA of the HEAD commit on the repository
@@ -158,6 +159,16 @@ public List<PushCommit> getCommits() {
             return commits;
         }
 
+        public GHRepository getRepository() {
+            return repository;
+        }
+
+        @Override
+        void wrapUp(GitHub root) {
+            if (repository!=null)
+                repository.wrap(root);
+        }
+
         /**
          * Commit in a push
          */

From 3bb7eb2e035d20e2499bcc07a1efe31c21383459 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sun, 15 Feb 2015 08:50:55 -0800
Subject: [PATCH 14/19] Added API to list contributors

---
 .../java/org/kohsuke/github/GHRepository.java | 21 +++++++++++++++++++
 .../org/kohsuke/github/RepositoryTest.java    | 19 +++++++++++++++++
 2 files changed, 40 insertions(+)

diff --git a/src/main/java/org/kohsuke/github/GHRepository.java b/src/main/java/org/kohsuke/github/GHRepository.java
index c1d29ccbdd..4707bb0aa8 100644
--- a/src/main/java/org/kohsuke/github/GHRepository.java
+++ b/src/main/java/org/kohsuke/github/GHRepository.java
@@ -1093,6 +1093,27 @@ public GHSubscription getSubscription() throws IOException {
         }
     }
 
+    public PagedIterable<Contributor> listContributors() throws IOException {
+        return new PagedIterable<Contributor>() {
+            public PagedIterator<Contributor> iterator() {
+                return new PagedIterator<Contributor>(root.retrieve().asIterator(getApiTailUrl("contributors"), Contributor[].class)) {
+                    @Override
+                    protected void wrapUp(Contributor[] page) {
+                        for (Contributor c : page)
+                            c.wrapUp(root);
+                    }
+                };
+            }
+        };
+    }
+
+    public static class Contributor extends GHUser {
+        private int contributions;
+
+        public int getContributions() {
+            return contributions;
+        }
+    }
 	
 	
 
diff --git a/src/test/java/org/kohsuke/github/RepositoryTest.java b/src/test/java/org/kohsuke/github/RepositoryTest.java
index 2a5b62baab..6862ace7be 100644
--- a/src/test/java/org/kohsuke/github/RepositoryTest.java
+++ b/src/test/java/org/kohsuke/github/RepositoryTest.java
@@ -1,6 +1,7 @@
 package org.kohsuke.github;
 
 import org.junit.Test;
+import org.kohsuke.github.GHRepository.Contributor;
 
 import java.io.IOException;
 
@@ -21,6 +22,24 @@ public void subscription() throws Exception {
         assertNull(r.getSubscription());
     }
 
+    @Test
+    public void listContributors() throws IOException {
+        GHRepository r = gitHub.getOrganization("stapler").getRepository("stapler");
+        int i=0;
+        boolean kohsuke = false;
+
+        for (Contributor c : r.listContributors()) {
+            System.out.println(c.getName());
+            assertTrue(c.getContributions()>0);
+            if (c.getLogin().equals("kohsuke"))
+                kohsuke = true;
+            if (i++ > 5)
+                break;
+        }
+
+        assertTrue(kohsuke);
+    }
+
     private GHRepository getRepository() throws IOException {
         return gitHub.getOrganization("github-api-test-org").getRepository("jenkins");
     }

From b898284821f2037daa6bdf2e3036bab508045d1c Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sun, 15 Feb 2015 08:56:29 -0800
Subject: [PATCH 15/19] Mentions thread-safety and state the goal.

Most of the objects are effectively immutable, so this should be an easy goal

Fixes issue #148.
---
 src/main/java/org/kohsuke/github/GitHub.java | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/kohsuke/github/GitHub.java b/src/main/java/org/kohsuke/github/GitHub.java
index 9b893040e0..c560227498 100644
--- a/src/main/java/org/kohsuke/github/GitHub.java
+++ b/src/main/java/org/kohsuke/github/GitHub.java
@@ -38,6 +38,7 @@
 import java.util.Date;
 import java.util.HashMap;
 import java.util.HashSet;
+import java.util.Hashtable;
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
@@ -53,6 +54,12 @@
 /**
  * Root of the GitHub API.
  *
+ * <h2>Thread safety</h2>
+ * <p>
+ * This library aims to be safe for use by multiple threads concurrently, although
+ * the library itself makes no attempt to control/serialize potentially conflicting
+ * operations to GitHub, such as updating & deleting a repository at the same time.
+ *
  * @author Kohsuke Kawaguchi
  */
 public class GitHub {
@@ -63,8 +70,8 @@ public class GitHub {
      */
     /*package*/ final String encodedAuthorization;
 
-    private final Map<String,GHUser> users = new HashMap<String, GHUser>();
-    private final Map<String,GHOrganization> orgs = new HashMap<String, GHOrganization>();
+    private final Map<String,GHUser> users = new Hashtable<String, GHUser>();
+    private final Map<String,GHOrganization> orgs = new Hashtable<String, GHOrganization>();
 
     private final String apiUrl;
 

From 15163ffde0d85b2404f1904aa2e5475582ed226d Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sun, 15 Feb 2015 08:56:57 -0800
Subject: [PATCH 16/19] Avoid multiple concurrent population

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

diff --git a/src/main/java/org/kohsuke/github/GHPerson.java b/src/main/java/org/kohsuke/github/GHPerson.java
index 7984d6d2cc..fa651bb681 100644
--- a/src/main/java/org/kohsuke/github/GHPerson.java
+++ b/src/main/java/org/kohsuke/github/GHPerson.java
@@ -36,7 +36,7 @@ public abstract class GHPerson extends GHObject {
      *
      * Depending on the original API call where this object is created, it may not contain everything.
      */
-    protected void populate() throws IOException {
+    protected synchronized void populate() throws IOException {
         if (created_at!=null)    return; // already populated
 
         root.retrieve().to(url, this);

From e631e46dd1782d8b5362982fcef39fb80cc8f83f Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sun, 15 Feb 2015 08:57:36 -0800
Subject: [PATCH 17/19] Exposed this method.

I'm generally against having these inter-object short cut methods
(in this case it's getOwner().getName() but oh well.)

Fixes issue #149
---
 src/main/java/org/kohsuke/github/GHRepository.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/java/org/kohsuke/github/GHRepository.java b/src/main/java/org/kohsuke/github/GHRepository.java
index 4707bb0aa8..8570a09bb1 100644
--- a/src/main/java/org/kohsuke/github/GHRepository.java
+++ b/src/main/java/org/kohsuke/github/GHRepository.java
@@ -285,7 +285,7 @@ protected void wrapUp(GHTag[] page) {
         };
     }
 
-    protected String getOwnerName() {
+    public String getOwnerName() {
         return owner.login;
     }
 

From c3a9f6f9f5ed64f6c6984ab7271870ca2926ad5c Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sun, 15 Feb 2015 09:06:04 -0800
Subject: [PATCH 18/19] Fixed NPE.

issue #152
---
 src/main/java/org/kohsuke/github/GHTag.java      |  2 ++
 src/test/java/org/kohsuke/github/CommitTest.java | 16 ++++++++++++++++
 2 files changed, 18 insertions(+)
 create mode 100644 src/test/java/org/kohsuke/github/CommitTest.java

diff --git a/src/main/java/org/kohsuke/github/GHTag.java b/src/main/java/org/kohsuke/github/GHTag.java
index 60d4566cc8..3a72407754 100644
--- a/src/main/java/org/kohsuke/github/GHTag.java
+++ b/src/main/java/org/kohsuke/github/GHTag.java
@@ -15,6 +15,8 @@ public class GHTag {
     /*package*/ GHTag wrap(GHRepository owner) {
         this.owner = owner;
         this.root = owner.root;
+        if (commit!=null)
+            commit.wrapUp(owner);
         return this;
     }
 
diff --git a/src/test/java/org/kohsuke/github/CommitTest.java b/src/test/java/org/kohsuke/github/CommitTest.java
new file mode 100644
index 0000000000..fff9084ebd
--- /dev/null
+++ b/src/test/java/org/kohsuke/github/CommitTest.java
@@ -0,0 +1,16 @@
+package org.kohsuke.github;
+
+import org.junit.Test;
+
+import java.io.IOException;
+
+/**
+ * @author Kohsuke Kawaguchi
+ */
+public class CommitTest extends AbstractGitHubApiTestBase {
+    @Test // issue 152
+    public void lastStatus() throws IOException {
+        GHTag t = gitHub.getRepository("stapler/stapler").listTags().iterator().next();
+        t.getCommit().getLastStatus();
+    }
+}

From 67280951ffc108935ee9c242b23230a5d89f2a5d Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Sun, 15 Feb 2015 09:12:23 -0800
Subject: [PATCH 19/19] [maven-release-plugin] prepare release github-api-1.62

---
 pom.xml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/pom.xml b/pom.xml
index 36c9f040a2..69ffb80db3 100644
--- a/pom.xml
+++ b/pom.xml
@@ -7,7 +7,7 @@
   </parent>
 
   <artifactId>github-api</artifactId>
-  <version>1.62-SNAPSHOT</version>
+  <version>1.62</version>
   <name>GitHub API for Java</name>
   <url>http://github-api.kohsuke.org/</url>
   <description>GitHub API for Java</description>
@@ -16,7 +16,7 @@
     <connection>scm:git:git@github.com/kohsuke/${project.artifactId}.git</connection>
     <developerConnection>scm:git:ssh://git@github.com/kohsuke/${project.artifactId}.git</developerConnection>
     <url>http://${project.artifactId}.kohsuke.org/</url>
-    <tag>HEAD</tag>
+    <tag>github-api-1.62</tag>
   </scm>
 
   <distributionManagement>