[maven-release-plugin] prepare release github-api-1.95

In particular, avoid the kind of addLabel() method that has lots of side
effect and do multiple things.

pom.xml

@@ -3,11 +3,11 @@
   <parent>
     <groupId>org.kohsuke</groupId>
     <artifactId>pom</artifactId>
-    <version>17</version>
+    <version>20</version>
   </parent>
 
   <artifactId>github-api</artifactId>
-  <version>1.93</version>
+  <version>1.95</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.93</tag>
+    <tag>github-api-1.95</tag>
   </scm>
 
   <distributionManagement>
@@ -36,6 +36,7 @@
     <plugins>
       <plugin>
         <artifactId>maven-surefire-plugin</artifactId>
+        <version>2.22.1</version>
         <configuration>
           <rerunFailingTestsCount>2</rerunFailingTestsCount>
         </configuration>

src/main/java/org/kohsuke/github/GHBranch.java

@@ -65,9 +65,8 @@ public class GHBranch {
         return GitHub.parseURL(protection_url);
     }
 
-    @Preview @Deprecated
     public GHBranchProtection getProtection() throws IOException {
-        return root.retrieve().withPreview(LOKI).to(protection_url, GHBranchProtection.class);
+        return root.retrieve().to(protection_url, GHBranchProtection.class).wrap(this);
     }
 
     /**
@@ -80,9 +79,8 @@ public class GHBranch {
     /**
      * Disables branch protection and allows anyone with push access to push changes.
      */
-    @Preview @Deprecated
     public void disableProtection() throws IOException {
-        new Requester(root).method("DELETE").withPreview(LOKI).to(protection_url);
+        new Requester(root).method("DELETE").to(protection_url);
     }
 
     /**

src/main/java/org/kohsuke/github/GHBranchProtection.java

@@ -3,27 +3,46 @@ package org.kohsuke.github;
 import com.fasterxml.jackson.annotation.JsonProperty;
 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 
+import static org.kohsuke.github.Previews.ZZZAX;
+
+import java.io.IOException;
 import java.util.Collection;
 
-@SuppressFBWarnings(value = { "UWF_UNWRITTEN_PUBLIC_OR_PROTECTED_FIELD", "UWF_UNWRITTEN_FIELD", "NP_UNWRITTEN_FIELD",
-		"URF_UNREAD_FIELD" }, justification = "JSON API")
+@SuppressFBWarnings(value = {"UWF_UNWRITTEN_PUBLIC_OR_PROTECTED_FIELD", "UWF_UNWRITTEN_FIELD", "NP_UNWRITTEN_FIELD",
+        "URF_UNREAD_FIELD"}, justification = "JSON API")
 public class GHBranchProtection {
-	@JsonProperty("enforce_admins")
-	private EnforceAdmins enforceAdmins;
+    private static final String REQUIRE_SIGNATURES_URI = "/required_signatures";
 
-	@JsonProperty("required_pull_request_reviews")
-	private RequiredReviews requiredReviews;
+    @JsonProperty("enforce_admins")
+    private EnforceAdmins enforceAdmins;
 
-	@JsonProperty("required_status_checks")
-	private RequiredStatusChecks requiredStatusChecks;
+    private GitHub root;
 
-	@JsonProperty
-	private Restrictions restrictions;
+    @JsonProperty("required_pull_request_reviews")
+    private RequiredReviews requiredReviews;
 
-	@JsonProperty
-	private String url;
-	
-	public EnforceAdmins getEnforceAdmins() {
+    @JsonProperty("required_status_checks")
+    private RequiredStatusChecks requiredStatusChecks;
+
+    @JsonProperty
+    private Restrictions restrictions;
+
+    @JsonProperty
+    private String url;
+
+    @Preview @Deprecated
+    public void enabledSignedCommits() throws IOException {
+        requester().method("POST")
+                .to(url + REQUIRE_SIGNATURES_URI, RequiredSignatures.class);
+    }
+
+    @Preview @Deprecated
+    public void disableSignedCommits() throws IOException {
+        requester().method("DELETE")
+                .to(url + REQUIRE_SIGNATURES_URI);
+    }
+
+    public EnforceAdmins getEnforceAdmins() {
         return enforceAdmins;
     }
 
@@ -31,6 +50,12 @@ public class GHBranchProtection {
         return requiredReviews;
     }
 
+    @Preview @Deprecated
+    public boolean getRequiredSignatures() throws IOException {
+        return requester().method("GET")
+                .to(url + REQUIRE_SIGNATURES_URI, RequiredSignatures.class).enabled;
+    }
+
     public RequiredStatusChecks getRequiredStatusChecks() {
         return requiredStatusChecks;
     }
@@ -43,12 +68,21 @@ public class GHBranchProtection {
         return url;
     }
 
-    public static class EnforceAdmins {
-		@JsonProperty
-		private boolean enabled;
+    GHBranchProtection wrap(GHBranch branch) {
+        this.root = branch.getRoot();
+        return this;
+    }
 
-		@JsonProperty
-		private String url;
+    private Requester requester() {
+        return new Requester(root).withPreview(ZZZAX);
+    }
+
+    public static class EnforceAdmins {
+        @JsonProperty
+        private boolean enabled;
+
+        @JsonProperty
+        private String url;
 
         public String getUrl() {
             return url;
@@ -57,20 +91,23 @@ public class GHBranchProtection {
         public boolean isEnabled() {
             return enabled;
         }
-	}
+    }
 
-	public static class RequiredReviews {
-		@JsonProperty("dismissal_restrictions")
-		private Restrictions dismissalRestriction; 
+    public static class RequiredReviews {
+        @JsonProperty("dismissal_restrictions")
+        private Restrictions dismissalRestriction;
 
-		@JsonProperty("dismiss_stale_reviews")
-		private boolean dismissStaleReviews;
+        @JsonProperty("dismiss_stale_reviews")
+        private boolean dismissStaleReviews;
 
-		@JsonProperty("require_code_owner_reviews")
-		private boolean requireCodeOwnerReviews;
+        @JsonProperty("require_code_owner_reviews")
+        private boolean requireCodeOwnerReviews;
 
-		@JsonProperty
-		private String url;
+        @JsonProperty("required_approving_review_count")
+        private int requiredReviewers;
+
+        @JsonProperty
+        private String url;
 
         public Restrictions getDismissalRestrictions() {
             return dismissalRestriction;
@@ -87,17 +124,37 @@ public class GHBranchProtection {
         public boolean isRequireCodeOwnerReviews() {
             return requireCodeOwnerReviews;
         }
-	}
 
-	public static class RequiredStatusChecks {
-		@JsonProperty
-		private Collection<String> contexts;
+        public int getRequiredReviewers() {
+            return requiredReviewers;
+        }
+    }
 
-		@JsonProperty
-		private boolean strict;
+    private static class RequiredSignatures {
+        @JsonProperty
+        private boolean enabled;
 
-		@JsonProperty
-		private String url;
+        @JsonProperty
+        private String url;
+
+        public String getUrl() {
+            return url;
+        }
+
+        public boolean isEnabled() {
+            return enabled;
+        }
+    }
+
+    public static class RequiredStatusChecks {
+        @JsonProperty
+        private Collection<String> contexts;
+
+        @JsonProperty
+        private boolean strict;
+
+        @JsonProperty
+        private String url;
 
         public Collection<String> getContexts() {
             return contexts;
@@ -110,23 +167,23 @@ public class GHBranchProtection {
         public boolean isRequiresBranchUpToDate() {
             return strict;
         }
-	}
+    }
 
-	public static class Restrictions {
-		@JsonProperty
-		private Collection<GHTeam> teams;
+    public static class Restrictions {
+        @JsonProperty
+        private Collection<GHTeam> teams;
 
-		@JsonProperty("teams_url")
-		private String teamsUrl;
+        @JsonProperty("teams_url")
+        private String teamsUrl;
 
-		@JsonProperty
-		private String url;
+        @JsonProperty
+        private String url;
 
-		@JsonProperty
-		private Collection<GHUser> users;
+        @JsonProperty
+        private Collection<GHUser> users;
 
-		@JsonProperty("users_url")
-		private String usersUrl;
+        @JsonProperty("users_url")
+        private String usersUrl;
 
         public Collection<GHTeam> getTeams() {
             return teams;
@@ -147,5 +204,5 @@ public class GHBranchProtection {
         public String getUsersUrl() {
             return usersUrl;
         }
-	}
+    }
 }

src/main/java/org/kohsuke/github/GHBranchProtectionBuilder.java

@@ -44,7 +44,11 @@ public class GHBranchProtectionBuilder {
     }
 
     public GHBranchProtectionBuilder dismissStaleReviews() {
-        getPrReviews().put("dismiss_stale_reviews", true);
+        return dismissStaleReviews(true);
+    }
+
+    public GHBranchProtectionBuilder dismissStaleReviews(boolean v) {
+        getPrReviews().put("dismiss_stale_reviews", v);
         return this;
     }
 
@@ -54,7 +58,8 @@ public class GHBranchProtectionBuilder {
                 .withNullable("required_pull_request_reviews", prReviews)
                 .withNullable("restrictions", restrictions)
                 .withNullable("enforce_admins", enforceAdmins)
-                .to(branch.getProtectionUrl().toString(), GHBranchProtection.class);
+                .to(branch.getProtectionUrl().toString(), GHBranchProtection.class)
+                .wrap(branch);
     }
 
     public GHBranchProtectionBuilder includeAdmins() {
@@ -66,6 +71,11 @@ public class GHBranchProtectionBuilder {
         return this;
     }
 
+    public GHBranchProtectionBuilder requiredReviewers(int v) {
+        getPrReviews().put("required_approving_review_count", v);
+        return this;
+    }
+
     public GHBranchProtectionBuilder requireBranchIsUpToDate() {
         return requireBranchIsUpToDate(true);
     }
@@ -89,6 +99,16 @@ public class GHBranchProtectionBuilder {
         return this;
     }
 
+    public GHBranchProtectionBuilder restrictReviewDismissals() {
+        getPrReviews();
+
+        if (!prReviews.containsKey("dismissal_restrictions")) {
+            prReviews.put("dismissal_restrictions", new Restrictions());
+        }
+
+        return this;
+    }
+
     public GHBranchProtectionBuilder restrictPushAccess() {
         getRestrictions();
         return this;
@@ -151,12 +171,7 @@ public class GHBranchProtectionBuilder {
     }
 
     private void addReviewRestriction(String restriction, boolean isTeam) {
-        getPrReviews();
-        
-        if (!prReviews.containsKey("dismissal_restrictions")) {
-            prReviews.put("dismissal_restrictions", new Restrictions());
-        }
-        
+        restrictReviewDismissals();
         Restrictions restrictions = (Restrictions) prReviews.get("dismissal_restrictions");
         
         if (isTeam) {
@@ -188,7 +203,7 @@ public class GHBranchProtectionBuilder {
     }
 
     private Requester requester() {
-        return new Requester(branch.getRoot()).withPreview(LOKI);
+        return new Requester(branch.getRoot()).withPreview(LUKE_CAGE);
     }
 
     private static class Restrictions {

src/main/java/org/kohsuke/github/GHContentBuilder.java

@@ -0,0 +1,76 @@
+package org.kohsuke.github;
+
+import org.apache.commons.codec.binary.Base64;
+
+import java.io.IOException;
+import java.io.UnsupportedEncodingException;
+
+/**
+ * Used to create/update content.
+ *
+ * <p>
+ * Call various methods to build up parameters, then call {@link #commit()} to make the change effective.
+ *
+ * @author Kohsuke Kawaguchi
+ * @see GHRepository#createContent()
+ */
+public final class GHContentBuilder {
+    private final GHRepository repo;
+    private final Requester req;
+    private String path;
+
+    GHContentBuilder(GHRepository repo) {
+        this.repo = repo;
+        this.req = new Requester(repo.root).method("PUT");
+    }
+
+    public GHContentBuilder path(String path) {
+        this.path = path;
+        req.with("path",path);
+        return this;
+    }
+
+    public GHContentBuilder branch(String branch) {
+        req.with("branch", branch);
+        return this;
+    }
+
+    /**
+     * Used when updating (but not creating a new content) to specify
+     * Thetblob SHA of the file being replaced.
+     */
+    public GHContentBuilder sha(String sha) {
+        req.with("sha", sha);
+        return this;
+    }
+
+    public GHContentBuilder content(byte[] content) {
+        req.with("content", Base64.encodeBase64String(content));
+        return this;
+    }
+
+    public GHContentBuilder content(String content) {
+        try {
+            return content(content.getBytes("UTF-8"));
+        } catch (UnsupportedEncodingException x) {
+            throw new AssertionError();
+        }
+    }
+
+    public GHContentBuilder message(String commitMessage) {
+        req.with("message", commitMessage);
+        return this;
+    }
+
+    /**
+     * Commits a new content.
+     */
+    public GHContentUpdateResponse commit() throws IOException {
+        GHContentUpdateResponse response = req.to(repo.getApiTailUrl("contents/" + path), GHContentUpdateResponse.class);
+
+        response.getContent().wrap(repo);
+        response.getCommit().wrapUp(repo);
+
+        return response;
+    }
+}

src/main/java/org/kohsuke/github/GHEventPayload.java

@@ -174,6 +174,51 @@ public abstract class GHEventPayload {
         }
     }
 
+    /**
+     * A Issue has been assigned, unassigned, labeled, unlabeled, opened, edited, milestoned, demilestoned, closed, or reopened.
+     *
+     * @see <a href="http://developer.github.com/v3/activity/events/types/#issueevent">authoritative source</a>
+     */
+    @SuppressFBWarnings(value = {"UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR", "NP_UNWRITTEN_FIELD" },
+            justification = "Constructed by JSON deserialization")
+    public static class Issue extends GHEventPayload {
+        private String action;
+        private GHIssue issue;
+        private GHRepository repository;
+
+        @SuppressFBWarnings(value = "UWF_UNWRITTEN_FIELD", justification = "Comes from JSON deserialization")
+        public String getAction() {
+            return action;
+        }
+
+        public GHIssue getIssue() {
+            return issue;
+        }
+
+        public void setIssue(GHIssue issue) {
+            this.issue = issue;
+        }
+
+        public GHRepository getRepository() {
+            return repository;
+        }
+
+        public void setRepository(GHRepository repository) {
+            this.repository = repository;
+        }
+
+        @Override
+        void wrapUp(GitHub root) {
+            super.wrapUp(root);
+            if (repository != null) {
+                repository.wrap(root);
+                issue.wrap(repository);
+            } else {
+                issue.wrap(root);
+            }
+        }
+    }
+
     /**
      * A comment was added to an issue
      *
@@ -716,6 +761,48 @@ public abstract class GHEventPayload {
         }
     }
 
+    /**
+     * A release was added to the repo
+     *
+     * @see <a href="http://developer.github.com/v3/activity/events/types/#releaseevent">authoritative source</a>
+     */
+    @SuppressFBWarnings(value = {"UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR", "NP_UNWRITTEN_FIELD" },
+            justification = "Constructed by JSON deserialization")
+    public static class Release extends GHEventPayload {
+        private String action;
+        private GHRelease release;
+        private GHRepository repository;
+
+        @SuppressFBWarnings(value = "UWF_UNWRITTEN_FIELD", justification = "Comes from JSON deserialization")
+        public String getAction() {
+            return action;
+        }
+
+        public GHRelease getRelease() {
+            return release;
+        }
+
+        public void setRelease(GHRelease release) {
+            this.release = release;
+        }
+
+        public GHRepository getRepository() {
+            return repository;
+        }
+
+        public void setRepository(GHRepository repository) {
+            this.repository = repository;
+        }
+
+        @Override
+        void wrapUp(GitHub root) {
+            super.wrapUp(root);
+            if (repository != null) {
+                repository.wrap(root);
+            }
+        }
+    }
+
     /**
      * A repository was created, deleted, made public, or made private.
      *

src/main/java/org/kohsuke/github/GHInvitation.java

@@ -0,0 +1,46 @@
+package org.kohsuke.github;
+
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
+
+import java.io.IOException;
+import java.net.URL;
+
+/**
+ * @see GitHub#getMyInvitations()
+ * @see GHRepository#listInvitations()
+ */
+@SuppressFBWarnings(value = {"UWF_UNWRITTEN_PUBLIC_OR_PROTECTED_FIELD", "UWF_UNWRITTEN_FIELD",
+    "NP_UNWRITTEN_FIELD", "UUF_UNUSED_FIELD"}, justification = "JSON API")
+public class GHInvitation extends GHObject {
+    /*package almost final*/ GitHub root;
+
+    private int id;
+    private GHRepository repository;
+    private GHUser invitee, inviter;
+    private String permissions;
+    private String html_url;
+
+    /*package*/ GHInvitation wrapUp(GitHub root) {
+        this.root = root;
+        return this;
+    }
+
+    /**
+     * Accept a repository invitation.
+     */
+    public void accept() throws IOException {
+        root.retrieve().method("PATCH").to("/user/repository_invitations/" + id);
+    }
+
+    /**
+     * Decline a repository invitation.
+     */
+    public void decline() throws IOException {
+        root.retrieve().method("DELETE").to("/user/repository_invitations/" + id);
+    }
+
+    @Override
+    public URL getHtmlUrl() {
+        return GitHub.parseURL(html_url);
+    }
+}

src/main/java/org/kohsuke/github/GHIssue.java

@@ -24,19 +24,21 @@
 
 package org.kohsuke.github;
 
+import static org.kohsuke.github.Previews.SQUIRREL_GIRL;
+
 import com.infradna.tool.bridge_method_injector.WithBridgeMethods;
 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
-
 import java.io.IOException;
 import java.net.URL;
+import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collection;
 import java.util.Collections;
 import java.util.Date;
+import java.util.HashSet;
 import java.util.List;
 import java.util.Locale;
-
-import static org.kohsuke.github.Previews.*;
+import java.util.Set;
 
 /**
  * Represents an issue on GitHub.
@@ -216,6 +218,67 @@ public class GHIssue extends GHObject implements Reactable{
         editIssue("labels",labels);
     }
 
+    /**
+     * Adds labels to the issue.
+     *
+     * @param names Names of the label
+     */
+    public void addLabels(String... names) throws IOException {
+        _addLabels(Arrays.asList(names));
+    }
+
+    public void addLabels(GHLabel... labels) throws IOException {
+        addLabels(Arrays.asList(labels));
+    }
+
+    public void addLabels(Collection<GHLabel> labels) throws IOException {
+        _addLabels(GHLabel.toNames(labels));
+    }
+
+    private void _addLabels(Collection<String> names) throws IOException {
+        List<String> newLabels = new ArrayList<String>();
+
+        for (GHLabel label : getLabels()) {
+            newLabels.add(label.getName());
+        }
+        for (String name : names) {
+            if (!newLabels.contains(name)) {
+                newLabels.add(name);
+            }
+        }
+        setLabels(newLabels.toArray(new String[0]));
+    }
+
+    /**
+     * Remove a given label by name from this issue.
+     */
+    public void removeLabels(String... names) throws IOException {
+        _removeLabels(Arrays.asList(names));
+    }
+
+    /**
+     * @see #removeLabels(String...)
+     */
+    public void removeLabels(GHLabel... labels) throws IOException {
+        removeLabels(Arrays.asList(labels));
+    }
+
+    public void removeLabels(Collection<GHLabel> labels) throws IOException {
+        _removeLabels(GHLabel.toNames(labels));
+    }
+
+    private void _removeLabels(Collection<String> names) throws IOException {
+        List<String> newLabels = new ArrayList<String>();
+
+        for (GHLabel l : getLabels()) {
+            if (!names.contains(l.getName())) {
+                newLabels.add(l.getName());
+            }
+        }
+
+        setLabels(newLabels.toArray(new String[0]));
+    }
+
     /**
      * Obtains all the comments associated with this issue.
      * 

src/main/java/org/kohsuke/github/GHLabel.java

@@ -1,6 +1,9 @@
 package org.kohsuke.github;
 
 import java.io.IOException;
+import java.util.ArrayList;
+import java.util.Collection;
+import java.util.List;
 
 /**
  * @author Kohsuke Kawaguchi
@@ -42,4 +45,12 @@ public class GHLabel {
     public void setColor(String newColor) throws IOException {
         repo.root.retrieve().method("PATCH").with("name", name).with("color", newColor).to(url);
     }
+
+    /*package*/ static Collection<String> toNames(Collection<GHLabel> labels) {
+        List<String> r = new ArrayList<String>();
+        for (GHLabel l : labels) {
+            r.add(l.getName());
+        }
+        return r;
+    }
 }

src/main/java/org/kohsuke/github/GHOrganization.java

@@ -50,7 +50,7 @@ public class GHOrganization extends GHPerson {
      * You use the returned builder to set various properties, then call {@link GHCreateRepositoryBuilder#create()}
      * to finally createa repository.
      */
-    public GHCreateRepositoryBuilder createRepository(String name) throws IOException {
+    public GHCreateRepositoryBuilder createRepository(String name) {
         return new GHCreateRepositoryBuilder(root,"/orgs/"+login+"/repos",name);
     }
 
@@ -269,7 +269,7 @@ public class GHOrganization extends GHPerson {
     public PagedIterable<GHRepository> listRepositories(final int pageSize) {
         return new PagedIterable<GHRepository>() {
             public PagedIterator<GHRepository> _iterator(int pageSize) {
-                return new PagedIterator<GHRepository>(root.retrieve().asIterator("/orgs/" + login + "/repos?per_page=" + pageSize, GHRepository[].class, pageSize)) {
+                return new PagedIterator<GHRepository>(root.retrieve().asIterator("/orgs/" + login + "/repos", GHRepository[].class, pageSize)) {
                     @Override
                     protected void wrapUp(GHRepository[] page) {
                         for (GHRepository c : page)
@@ -277,7 +277,7 @@ public class GHOrganization extends GHPerson {
                     }
                 };
             }
-        };
+        }.withPageSize(pageSize);
     }
 
     /**

src/main/java/org/kohsuke/github/GHPerson.java

@@ -81,7 +81,7 @@ public abstract class GHPerson extends GHObject {
     public PagedIterable<GHRepository> listRepositories(final int pageSize) {
         return new PagedIterable<GHRepository>() {
             public PagedIterator<GHRepository> _iterator(int pageSize) {
-                return new PagedIterator<GHRepository>(root.retrieve().asIterator("/users/" + login + "/repos?per_page=" + pageSize, GHRepository[].class, pageSize)) {
+                return new PagedIterator<GHRepository>(root.retrieve().asIterator("/users/" + login + "/repos", GHRepository[].class, pageSize)) {
                     @Override
                     protected void wrapUp(GHRepository[] page) {
                         for (GHRepository c : page)
@@ -89,7 +89,7 @@ public abstract class GHPerson extends GHObject {
                     }
                 };
             }
-        };
+        }.withPageSize(pageSize);
     }
 
     /**
@@ -108,7 +108,7 @@ public abstract class GHPerson extends GHObject {
     public synchronized Iterable<List<GHRepository>> iterateRepositories(final int pageSize) {
         return new Iterable<List<GHRepository>>() {
             public Iterator<List<GHRepository>> iterator() {
-                final Iterator<GHRepository[]> pager = root.retrieve().asIterator("/users/" + login + "/repos?per_page="+pageSize,GHRepository[].class, pageSize);
+                final Iterator<GHRepository[]> pager = root.retrieve().asIterator("/users/" + login + "/repos",GHRepository[].class, pageSize);
 
                 return new Iterator<List<GHRepository>>() {
                     public boolean hasNext() {

src/main/java/org/kohsuke/github/GHPullRequest.java

@@ -28,6 +28,7 @@ import java.io.IOException;
 import java.net.URL;
 import java.util.Arrays;
 import java.util.Collection;
+import java.util.Collections;
 import java.util.Date;
 import java.util.List;
 
@@ -58,6 +59,9 @@ public class GHPullRequest extends GHIssue {
     private int changed_files;
     private String merge_commit_sha;
 
+    // pull request reviewers
+    private GHUser[] requested_reviewers;
+
     /**
      * GitHub doesn't return some properties of {@link GHIssue} when requesting the GET on the 'pulls' API
      * route as opposed to 'issues' API route. This flag remembers whether we made the GET call on the 'issues' route
@@ -76,6 +80,7 @@ public class GHPullRequest extends GHIssue {
         if (base != null) base.wrapUp(root);
         if (head != null) head.wrapUp(root);
         if (merged_by != null) merged_by.wrapUp(root);
+        if (requested_reviewers != null) GHUser.wrap(requested_reviewers, root);
         return this;
     }
 
@@ -219,6 +224,11 @@ public class GHPullRequest extends GHIssue {
         return merge_commit_sha;
     }
 
+    public List<GHUser> getRequestedReviewers() throws IOException {
+        populate();
+        return Collections.unmodifiableList(Arrays.asList(requested_reviewers));
+    }
+
     /**
      * Fully populate the data by retrieving missing data.
      *

src/main/java/org/kohsuke/github/GHRelease.java

@@ -3,6 +3,7 @@ package org.kohsuke.github;
 import java.io.File;
 import java.io.FileInputStream;
 import java.io.IOException;
+import java.io.InputStream;
 import java.net.URL;
 import java.util.Arrays;
 import java.util.Date;
@@ -125,12 +126,21 @@ public class GHRelease extends GHObject {
      * handling of the HTTP requests to github's API.
          */
     public GHAsset uploadAsset(File file, String contentType) throws IOException {
+        FileInputStream s = new FileInputStream(file);
+        try {
+            return uploadAsset(file.getName(), s, contentType);
+        } finally {
+            s.close();
+        }
+    }
+    
+    public GHAsset uploadAsset(String filename, InputStream stream, String contentType) throws IOException {
         Requester builder = new Requester(owner.root);
 
         String url = format("https://uploads.github.com%s/releases/%d/assets?name=%s",
-                owner.getApiTailUrl(""), getId(), file.getName());
+                owner.getApiTailUrl(""), getId(), filename);
         return builder.contentType(contentType)
-                .with(new FileInputStream(file))
+                .with(stream)
                 .to(url, GHAsset.class).wrap(this);
     }
 

src/main/java/org/kohsuke/github/GHRepository.java

@@ -26,7 +26,6 @@ package org.kohsuke.github;
 import com.fasterxml.jackson.annotation.JsonProperty;
 import com.infradna.tool.bridge_method_injector.WithBridgeMethods;
 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
-import org.apache.commons.codec.binary.Base64;
 import org.apache.commons.lang3.StringUtils;
 
 import java.io.FileNotFoundException;
@@ -35,7 +34,6 @@ import java.io.InputStream;
 import java.io.InputStreamReader;
 import java.io.InterruptedIOException;
 import java.io.Reader;
-import java.io.UnsupportedEncodingException;
 import java.net.URL;
 import java.util.AbstractSet;
 import java.util.ArrayList;
@@ -50,6 +48,7 @@ import java.util.List;
 import java.util.Map;
 import java.util.Set;
 import java.util.TreeMap;
+import java.util.WeakHashMap;
 
 import static java.util.Arrays.*;
 import static org.kohsuke.github.Previews.*;
@@ -76,15 +75,15 @@ public class GHRepository extends GHObject {
 
     private String git_url, ssh_url, clone_url, svn_url, mirror_url;
     private GHUser owner;   // not fully populated. beware.
-    private boolean has_issues, has_wiki, fork, has_downloads, has_pages;
+    private boolean has_issues, has_wiki, fork, has_downloads, has_pages, archived;
     @JsonProperty("private")
     private boolean _private;
     private int forks_count, stargazers_count, watchers_count, size, open_issues_count, subscribers_count;
     private String pushed_at;
-    private Map<Integer,GHMilestone> milestones = new HashMap<Integer, GHMilestone>();
+    private Map<Integer,GHMilestone> milestones = new WeakHashMap<Integer, GHMilestone>();
 
     private String default_branch,language;
-    private Map<String,GHCommit> commits = new HashMap<String, GHCommit>();
+    private Map<String,GHCommit> commits = new WeakHashMap<String, GHCommit>();
 
     @SkipFromToString
     private GHRepoPermission permissions;
@@ -310,6 +309,22 @@ public class GHRepository extends GHObject {
     public List<GHRelease> getReleases() throws IOException {
         return listReleases().asList();
     }
+
+    public GHRelease getRelease(long id) throws IOException {
+        try {
+            return root.retrieve().to(getApiTailUrl("releases/" + id), GHRelease.class).wrap(this);
+        } catch (FileNotFoundException e) {
+            return null; // no release for this id
+        }
+    }
+
+    public GHRelease getReleaseByTagName(String tag) throws IOException {
+        try {
+            return root.retrieve().to(getApiTailUrl("releases/tags/" + tag), GHRelease.class).wrap(this);
+        } catch (FileNotFoundException e) {
+            return null; // no release for this tag
+        }
+    }
     
     public GHRelease getLatestRelease() throws IOException {
         try {
@@ -379,6 +394,10 @@ public class GHRepository extends GHObject {
         return fork;
     }
 
+    public boolean isArchived() {
+        return archived;
+    }
+
     /**
      * Returns the number of all forks of this repository.
      * This not only counts direct forks, but also forks of forks, and so on.
@@ -815,7 +834,9 @@ public class GHRepository extends GHObject {
             public PagedIterator<GHRef> _iterator(int pageSize) {
                 return new PagedIterator<GHRef>(root.retrieve().asIterator(url, GHRef[].class, pageSize)) {
                     protected void wrapUp(GHRef[] page) {
-                        // no-op
+                        for(GHRef p: page) {
+                            p.wrap(root);
+                        }
                     }
                 };
             }
@@ -1137,6 +1158,22 @@ public class GHRepository extends GHObject {
                 .to(getApiTailUrl("labels"), GHLabel.class).wrapUp(this);
     }
 
+    /**
+     * Lists all the invitations.
+     */
+    public PagedIterable<GHInvitation> listInvitations() {
+        return new PagedIterable<GHInvitation>() {
+            public PagedIterator<GHInvitation> _iterator(int pageSize) {
+                return new PagedIterator<GHInvitation>(root.retrieve().asIterator(String.format("/repos/%s/%s/invitations", getOwnerName(), name), GHInvitation[].class, pageSize)) {
+                    protected void wrapUp(GHInvitation[] page) {
+                        for (GHInvitation c : page)
+                            c.wrapUp(root);
+                    }
+                };
+            }
+        };
+    }
+
     /**
      * Lists all the subscribers (aka watchers.)
      *
@@ -1306,7 +1343,7 @@ public class GHRepository extends GHObject {
      */
     public Map<String,GHBranch> getBranches() throws IOException {
         Map<String,GHBranch> r = new TreeMap<String,GHBranch>();
-        for (GHBranch p : root.retrieve().withPreview(LOKI).to(getApiTailUrl("branches"), GHBranch[].class)) {
+        for (GHBranch p : root.retrieve().to(getApiTailUrl("branches"), GHBranch[].class)) {
             p.wrap(this);
             r.put(p.getName(),p);
         }
@@ -1314,7 +1351,7 @@ public class GHRepository extends GHObject {
     }
 
     public GHBranch getBranch(String name) throws IOException {
-        return root.retrieve().withPreview(LOKI).to(getApiTailUrl("branches/"+name),GHBranch.class).wrap(this);
+        return root.retrieve().to(getApiTailUrl("branches/"+name),GHBranch.class).wrap(this);
     }
 
     /**
@@ -1394,41 +1431,43 @@ public class GHRepository extends GHObject {
         return requester.to(getApiTailUrl("readme"), GHContent.class).wrap(this);
     }
 
+    /**
+     * Creates a new content, or update an existing content.
+     */
+    public GHContentBuilder createContent() {
+        return new GHContentBuilder(this);
+    }
+
+    /**
+     * Use {@link #createContent()}.
+     */
+    @Deprecated
     public GHContentUpdateResponse createContent(String content, String commitMessage, String path) throws IOException {
-        return createContent(content, commitMessage, path, null);
+        return createContent().content(content).message(commitMessage).path(path).commit();
     }
 
+    /**
+     * Use {@link #createContent()}.
+     */
+    @Deprecated
     public GHContentUpdateResponse createContent(String content, String commitMessage, String path, String branch) throws IOException {
-        final byte[] payload;
-        try {
-            payload = content.getBytes("UTF-8");
-        } catch (UnsupportedEncodingException ex) {
-            throw (IOException) new IOException("UTF-8 encoding is not supported").initCause(ex);
-        }
-        return createContent(payload, commitMessage, path, branch);
+        return createContent().content(content).message(commitMessage).path(path).branch(branch).commit();
     }
 
+    /**
+     * Use {@link #createContent()}.
+     */
+    @Deprecated
     public GHContentUpdateResponse createContent(byte[] contentBytes, String commitMessage, String path) throws IOException {
-        return createContent(contentBytes, commitMessage, path, null);
+        return createContent().content(contentBytes).message(commitMessage).path(path).commit();
     }
 
+    /**
+     * Use {@link #createContent()}.
+     */
+    @Deprecated
     public GHContentUpdateResponse createContent(byte[] contentBytes, String commitMessage, String path, String branch) throws IOException {
-        Requester requester = new Requester(root)
-            .with("path", path)
-            .with("message", commitMessage)
-            .with("content", Base64.encodeBase64String(contentBytes))
-            .method("PUT");
-
-        if (branch != null) {
-            requester.with("branch", branch);
-        }
-
-        GHContentUpdateResponse response = requester.to(getApiTailUrl("contents/" + path), GHContentUpdateResponse.class);
-
-        response.getContent().wrap(this);
-        response.getCommit().wrapUp(this);
-
-        return response;
+        return createContent().content(contentBytes).message(commitMessage).path(path).branch(branch).commit();
     }
 
     public GHMilestone createMilestone(String title, String description) throws IOException {

src/main/java/org/kohsuke/github/GHRepositorySearchBuilder.java

@@ -55,6 +55,10 @@ public class GHRepositorySearchBuilder extends GHSearchBuilder<GHRepository> {
         return q("stars:"+v);
     }
 
+    public GHRepositorySearchBuilder topic(String v) {
+        return q("topic:"+v);
+    }
+
     public GHRepositorySearchBuilder order(GHDirection v) {
         req.with("order",v);
         return this;

src/main/java/org/kohsuke/github/GitHub.java

@@ -530,6 +530,15 @@ public class GitHub {
     }
 
     /**
+     * Gets complete list of open invitations for current user.
+     */
+    public List<GHInvitation> getMyInvitations() throws IOException {
+        GHInvitation[] invitations = retrieve().to("/user/repository_invitations", GHInvitation[].class);
+        for (GHInvitation i : invitations) {
+            i.wrapUp(this);
+        }
+        return Arrays.asList(invitations);
+    }
 
     /**
      * This method returns a shallowly populated organizations.

src/main/java/org/kohsuke/github/Previews.java

@@ -4,8 +4,9 @@ package org.kohsuke.github;
  * @author Kohsuke Kawaguchi
  */
 /*package*/ class Previews {
-    static final String LOKI = "application/vnd.github.loki-preview+json";
+    static final String LUKE_CAGE = "application/vnd.github.luke-cage-preview+json";
     static final String DRAX = "application/vnd.github.drax-preview+json";
     static final String SQUIRREL_GIRL = "application/vnd.github.squirrel-girl-preview";
     static final String CLOAK = "application/vnd.github.cloak-preview";
+    static final String ZZZAX = "application/vnd.github.zzzax-preview+json";
 }

src/test/java/org/kohsuke/github/GHBranchProtectionTest.java

@@ -32,6 +32,12 @@ public class GHBranchProtectionTest extends AbstractGitHubApiTestBase {
         branch = repo.getBranch(BRANCH);
 
         if (branch.isProtected()) {
+            GHBranchProtection protection = branch.getProtection();
+            if (protection.getRequiredSignatures()) {
+                protection.disableSignedCommits();
+            }
+
+            assertFalse(protection.getRequiredSignatures());
             branch.disableProtection();
         }
 
@@ -47,6 +53,7 @@ public class GHBranchProtectionTest extends AbstractGitHubApiTestBase {
                 .requireBranchIsUpToDate()
                 .requireCodeOwnReviews()
                 .dismissStaleReviews()
+                .requiredReviewers(2)
                 .includeAdmins()
                 .enable();
 
@@ -59,6 +66,7 @@ public class GHBranchProtectionTest extends AbstractGitHubApiTestBase {
         assertNotNull(requiredReviews);
         assertTrue(requiredReviews.isDismissStaleReviews());
         assertTrue(requiredReviews.isRequireCodeOwnerReviews());
+        assertEquals(2, requiredReviews.getRequiredReviewers());
         
         EnforceAdmins enforceAdmins = protection.getEnforceAdmins();
         assertNotNull(enforceAdmins);
@@ -79,4 +87,17 @@ public class GHBranchProtectionTest extends AbstractGitHubApiTestBase {
         
        assertNotNull(protection.getRequiredReviews());
     }
+
+    @Test
+    public void testSignedCommits() throws Exception {
+        GHBranchProtection protection = branch.enableProtection().enable();
+
+        assertFalse(protection.getRequiredSignatures());
+
+        protection.enabledSignedCommits();
+        assertTrue(protection.getRequiredSignatures());
+
+        protection.disableSignedCommits();
+        assertFalse(protection.getRequiredSignatures());
+    }
 }

src/test/java/org/kohsuke/github/GHContentIntegrationTest.java

@@ -53,7 +53,8 @@ public class GHContentIntegrationTest extends AbstractGitHubApiTestBase {
 
     @Test
     public void testCRUDContent() throws Exception {
-        GHContentUpdateResponse created = repo.createContent("this is an awesome file I created\n", "Creating a file for integration tests.", createdFilename);
+        GHContentUpdateResponse created =
+                repo.createContent("this is an awesome file I created\n", "Creating a file for integration tests.", createdFilename);
         GHContent createdContent = created.getContent();
 
         assertNotNull(created.getCommit());

src/test/java/org/kohsuke/github/GHEventPayloadTest.java

@@ -120,9 +120,19 @@ public class GHEventPayloadTest {
         assertThat(event.getSender().getLogin(), is("baxterthehacker"));
     }
 
-// TODO implement support classes and write test
-//    @Test
-//    public void issues() throws Exception {}
+    @Test
+    public void issues() throws Exception {
+        GHEventPayload.Issue event = GitHub.offline().parseEventPayload(payload.asReader(),GHEventPayload.Issue.class);
+        assertThat(event.getAction(),is("opened"));
+        assertThat(event.getIssue().getNumber(), is(2));
+        assertThat(event.getIssue().getTitle(), is("Spelling error in the README file"));
+        assertThat(event.getIssue().getState(), is(GHIssueState.OPEN));
+        assertThat(event.getIssue().getLabels().size(), is(1));
+        assertThat(event.getIssue().getLabels().iterator().next().getName(), is("bug"));
+        assertThat(event.getRepository().getName(), is("public-repo"));
+        assertThat(event.getRepository().getOwner().getLogin(), is("baxterthehacker"));
+        assertThat(event.getSender().getLogin(), is("baxterthehacker"));
+    }
 
 // TODO implement support classes and write test
 //    @Test

src/test/java/org/kohsuke/github/PullRequestTest.java

@@ -105,6 +105,7 @@ public class PullRequestTest extends AbstractGitHubApiTestBase {
         String name = rnd.next();
         GHRef masterRef = getRepository().getRef("heads/master");
         GHRef branchRef = getRepository().createRef("refs/heads/" + name, masterRef.getObject().getSha());
+
         getRepository().createContent(name, name, name, name);
         Thread.sleep(1000);
         GHPullRequest p = getRepository().createPullRequest(name, name, "master", "## test squash");
@@ -112,6 +113,27 @@ public class PullRequestTest extends AbstractGitHubApiTestBase {
         p.merge("squash merge", null, GHPullRequest.MergeMethod.SQUASH);
         branchRef.delete();
     }
+    @Test
+    public void testUpdateContentSquashMerge() throws Exception {
+        String name = rnd.next();
+        GHRef masterRef = getRepository().getRef("heads/master");
+        GHRef branchRef = getRepository().createRef("refs/heads/" + name, masterRef.getObject().getSha());
+
+        GHContentUpdateResponse response = getRepository().createContent(name, name, name, name);
+        Thread.sleep(1000);
+
+        getRepository().createContent()
+                .content(name + name)
+                .path(name)
+                .branch(name)
+                .message(name)
+                .sha(response.getContent().getSha())
+                .commit();
+        GHPullRequest p = getRepository().createPullRequest(name, name, "master", "## test squash");
+        Thread.sleep(1000);
+        p.merge("squash merge", null, GHPullRequest.MergeMethod.SQUASH);
+        branchRef.delete();
+    }
 
     @Test
     // Requires push access to the test repo to pass

src/test/java/org/kohsuke/github/RepositoryTest.java

@@ -94,6 +94,36 @@ public class RepositoryTest extends AbstractGitHubApiTestBase {
 		}
 	}
 
+    @Test public void listReleases() throws IOException {
+        PagedIterable<GHRelease> releases = gitHub.getOrganization("github").getRepository("hub").listReleases();
+        assertTrue(releases.iterator().hasNext());
+    }
+
+    @Test
+    public void getReleaseExists() throws IOException {
+        GHRelease release = gitHub.getOrganization("github").getRepository("hub").getRelease(6839710);
+        assertEquals("v2.3.0-pre10", release.getTagName());
+    }
+
+    @Test
+    public void getReleaseDoesNotExist() throws IOException {
+        GHRelease release = gitHub.getOrganization("github").getRepository("hub").getRelease(Long.MAX_VALUE);
+        assertNull(release);
+    }
+
+    @Test
+    public void getReleaseByTagNameExists() throws IOException {
+        GHRelease release = gitHub.getOrganization("github").getRepository("hub").getReleaseByTagName("v2.3.0-pre10");
+        assertNotNull(release);
+        assertEquals("v2.3.0-pre10", release.getTagName());
+    }
+
+    @Test
+    public void getReleaseByTagNameDoesNotExist() throws IOException {
+        GHRelease release = getRepository().getReleaseByTagName("foo-bar-baz");
+        assertNull(release);
+    }
+
     private GHRepository getRepository() throws IOException {
         return gitHub.getOrganization("github-api-test-org").getRepository("jenkins");
     }

src/test/java/org/kohsuke/github/UserTest.java

@@ -14,17 +14,17 @@ public class UserTest extends AbstractGitHubApiTestBase {
     public void listFollowsAndFollowers() throws IOException {
         GHUser u = gitHub.getUser("rtyler");
         assertNotEquals(
-            count50(u.listFollowers()),
-            count50(u.listFollows()));
+            count30(u.listFollowers()),
+            count30(u.listFollows()));
     }
 
-    private Set<GHUser> count50(PagedIterable<GHUser> l) {
+    private Set<GHUser> count30(PagedIterable<GHUser> l) {
         Set<GHUser> users = new HashSet<GHUser>();
         PagedIterator<GHUser> itr = l.iterator();
-        for (int i=0; i<50 && itr.hasNext(); i++) {
+        for (int i=0; i<30 && itr.hasNext(); i++) {
             users.add(itr.next());
         }
-        assertEquals(50, users.size());
+        assertEquals(30, users.size());
         return users;
     }
 }