Merge branch 'master' into feat/add-ot-support

Author: Abhi347

pom.xml

@@ -3,7 +3,7 @@
     <modelVersion>4.0.0</modelVersion>
 
     <artifactId>github-client</artifactId>
-    <version>0.3.3-SNAPSHOT</version>
+    <version>0.3.4-SNAPSHOT</version>
 
     <parent>
         <groupId>com.spotify</groupId>
@@ -67,7 +67,7 @@
     <properties>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
-        <project.build.outputTimestamp>1726146382</project.build.outputTimestamp>
+        <project.build.outputTimestamp>1737127111</project.build.outputTimestamp>
         <spotbugs.excludeFilterFile>spotbugsexclude.xml</spotbugs.excludeFilterFile>
         <checkstyle.violationSeverity>error</checkstyle.violationSeverity>
         <checkstyle.config.location>checkstyle.xml</checkstyle.config.location>

src/main/java/com/spotify/github/jackson/Json.java

@@ -20,7 +20,7 @@
 
 package com.spotify.github.jackson;
 
-import static com.fasterxml.jackson.databind.PropertyNamingStrategy.SNAKE_CASE;
+import com.fasterxml.jackson.databind.PropertyNamingStrategies;
 import static java.util.Objects.isNull;
 
 import com.fasterxml.jackson.annotation.JsonInclude;
@@ -248,6 +248,6 @@ private static class DefaultMapper {
             .enable(SerializationFeature.WRITE_DATES_WITH_ZONE_ID)
             .enable(DeserializationFeature.FAIL_ON_NULL_FOR_PRIMITIVES)
             .setSerializationInclusion(JsonInclude.Include.NON_NULL)
-            .setPropertyNamingStrategy(SNAKE_CASE);
+            .setPropertyNamingStrategy(PropertyNamingStrategies.SNAKE_CASE);
   }
 }

src/main/java/com/spotify/github/v3/activity/events/CreateEvent.java

@@ -47,7 +47,6 @@ public interface CreateEvent extends BaseEvent {
   String masterBranch();
 
   /** The repository's current description. */
-  @Nullable
   Optional<String> description();
 
   /** No doc found on github - Usually is "user". */

src/main/java/com/spotify/github/v3/activity/events/PushEvent.java

@@ -93,7 +93,6 @@ public interface PushEvent {
   List<PushCommit> commits();
 
   /** The push commit object of the most recent commit on ref after the push. */
-  @Nullable
   Optional<PushCommit> headCommit();
 
   /** Pusher */

src/main/java/com/spotify/github/v3/activity/events/StatusEvent.java

@@ -66,7 +66,6 @@ public interface StatusEvent extends BaseEvent, UpdateTracking {
   String context();
 
   /** The optional human-readable description added to the status. */
-  @Nullable
   Optional<String> description();
 
   /** The new state. Can be pending, success, failure, or error. */

src/main/java/com/spotify/github/v3/comment/Comment.java

@@ -60,6 +60,7 @@ public interface Comment extends UpdateTracking {
    *
    * @deprecated Use {@link #position()} instead
    */
+  @Deprecated
   Optional<Integer> line();
 
   /** Relative path of the file to comment on. */

src/main/java/com/spotify/github/v3/exceptions/GithubException.java

@@ -22,6 +22,8 @@
 
 /** Common github exception */
 public class GithubException extends RuntimeException {
+  private static final long serialVersionUID = 1L;
+
 
   /**
    * C'tor for setting a message

src/main/java/com/spotify/github/v3/exceptions/ReadOnlyRepositoryException.java

@@ -25,6 +25,8 @@
 
 /** The Read only repository exception. */
 public class ReadOnlyRepositoryException extends RequestNotOkException {
+  private static final long serialVersionUID = 1L;
+
   /**
    * Instantiates a new Read only repository exception.
    *

src/main/java/com/spotify/github/v3/exceptions/RequestNotOkException.java

@@ -39,6 +39,8 @@
 
 /** HTTP response with non-200 StatusCode. */
 public class RequestNotOkException extends GithubException {
+  private static final long serialVersionUID = 1L;
+
 
   private final int statusCode;
   private final String method;

src/main/java/com/spotify/github/v3/issues/Issue.java

@@ -48,11 +48,9 @@ public interface Issue extends CloseTracking {
   URI url();
 
   /** Events URL. */
-  @Nullable
   Optional<URI> eventsUrl();
 
   /** Repository URL. */
-  @Nullable
   Optional<URI> repositoryUrl();
 
   /** Labels URL template. */
@@ -80,7 +78,6 @@ public interface Issue extends CloseTracking {
   String title();
 
   /** The contents of the issue. */
-  @Nullable
   Optional<String> body();
 
   /** User. */