Add default values for options (#553)

Author: loonydev

github4s/src/main/scala/github4s/Decoders.scala

@@ -222,13 +222,13 @@ object Decoders {
 
   implicit val decodeStarredRepository: Decoder[StarredRepository] =
     Decoder[Repository]
-      .map(StarredRepository(None, _))
+      .map(StarredRepository(_))
       .or(
         Decoder.instance(c =>
           for {
             starred_at <- c.downField("starred_at").as[String]
             repo       <- c.downField("repo").as[Repository]
-          } yield StarredRepository(Some(starred_at), repo)
+          } yield StarredRepository(repo, Some(starred_at))
         )
       )
 
@@ -277,13 +277,13 @@ object Decoders {
 
   implicit val decodeStargazer: Decoder[Stargazer] =
     decoderUser
-      .map(Stargazer(None, _))
+      .map(Stargazer(_))
       .or(
         Decoder.instance(c =>
           for {
             starred_at <- c.downField("starred_at").as[String]
             user       <- c.downField("user").as[User]
-          } yield Stargazer(Some(starred_at), user)
+          } yield Stargazer(user, Some(starred_at))
         )
       )
 

github4s/src/main/scala/github4s/domain/Activity.scala

@@ -19,10 +19,10 @@ package github4s.domain
 final case class Subscription(
     subscribed: Boolean,
     ignored: Boolean,
-    reason: Option[String],
     created_at: String,
     url: String,
-    thread_url: String
+    thread_url: String,
+    reason: Option[String] = None
 )
 
 final case class SubscriptionRequest(
@@ -31,11 +31,11 @@ final case class SubscriptionRequest(
 )
 
 final case class Stargazer(
-    starred_at: Option[String],
-    user: User
+    user: User,
+    starred_at: Option[String] = None
 )
 
 final case class StarredRepository(
-    starred_at: Option[String],
-    repo: Repository
+    repo: Repository,
+    starred_at: Option[String] = None
 )

github4s/src/main/scala/github4s/domain/GitData.scala

@@ -71,29 +71,29 @@ final case class TreeDataResult(
     path: String,
     mode: String,
     `type`: String,
-    size: Option[Int],
     sha: String,
-    url: String
+    url: String,
+    size: Option[Int] = None
 )
 
 final case class NewCommitRequest(
     message: String,
     tree: String,
     parents: List[String],
-    author: Option[RefAuthor]
+    author: Option[RefAuthor] = None
 )
 
 final case class BlobContent(
-    content: Option[String],
-    encoding: Option[String],
     url: String,
     sha: String,
-    size: Int
+    size: Int,
+    content: Option[String] = None,
+    encoding: Option[String] = None
 )
 
-final case class NewBlobRequest(content: String, encoding: Option[String])
+final case class NewBlobRequest(content: String, encoding: Option[String] = None)
 
-final case class NewTreeRequest(base_tree: Option[String], tree: List[TreeData])
+final case class NewTreeRequest(tree: List[TreeData], base_tree: Option[String] = None)
 
 final case class CreateReferenceRequest(ref: String, sha: String)
 
@@ -104,5 +104,5 @@ final case class NewTagRequest(
     message: String,
     `object`: String,
     `type`: String,
-    tagger: Option[RefAuthor]
+    tagger: Option[RefAuthor] = None
 )

github4s/src/main/scala/github4s/domain/Issue.scala

@@ -21,7 +21,6 @@ import java.time.ZonedDateTime
 final case class Issue(
     id: Long,
     title: String,
-    body: Option[String],
     url: String,
     repository_url: String,
     labels_url: String,
@@ -30,31 +29,32 @@ final case class Issue(
     html_url: String,
     number: Int,
     state: String,
-    user: Option[User],
-    assignee: Option[User],
     labels: List[Label] = List.empty,
-    locked: Option[Boolean],
     comments: Int,
-    pull_request: Option[IssuePullRequest],
-    closed_at: Option[String],
     created_at: String,
-    updated_at: String
+    updated_at: String,
+    body: Option[String] = None,
+    user: Option[User] = None,
+    assignee: Option[User] = None,
+    locked: Option[Boolean] = None,
+    pull_request: Option[IssuePullRequest] = None,
+    closed_at: Option[String] = None
 )
 
 final case class Label(
-    id: Option[Long],
     name: String,
-    description: Option[String],
-    url: Option[String],
     color: String,
-    default: Option[Boolean]
+    id: Option[Long] = None,
+    description: Option[String] = None,
+    url: Option[String] = None,
+    default: Option[Boolean] = None
 )
 
 final case class IssuePullRequest(
-    url: Option[String],
-    html_url: Option[String],
-    diff_url: Option[String],
-    patch_url: Option[String]
+    url: Option[String] = None,
+    html_url: Option[String] = None,
+    diff_url: Option[String] = None,
+    patch_url: Option[String] = None
 )
 
 final case class SearchIssuesResult(
@@ -66,28 +66,28 @@ final case class SearchIssuesResult(
 final case class NewIssueRequest(
     title: String,
     body: String,
-    milestone: Option[Int],
     labels: List[String],
-    assignees: List[String]
+    assignees: List[String],
+    milestone: Option[Int] = None
 )
 
 final case class EditIssueRequest(
     state: String,
     title: String,
     body: String,
-    milestone: Option[Int],
     labels: List[String],
-    assignees: List[String]
+    assignees: List[String],
+    milestone: Option[Int] = None
 )
 
 final case class Comment(
     id: Long,
     url: String,
     html_url: String,
     body: String,
-    user: Option[User],
     created_at: String,
-    updated_at: String
+    updated_at: String,
+    user: Option[User] = None
 )
 
 final case class CommentData(body: String)
@@ -107,13 +107,13 @@ final case class Milestone(
     closed_issues: Int,
     created_at: String,
     updated_at: String,
-    closed_at: Option[String],
-    due_on: Option[String]
+    closed_at: Option[String] = None,
+    due_on: Option[String] = None
 )
 
 final case class MilestoneData(
     title: String,
-    state: Option[String],
-    description: Option[String],
-    due_on: Option[ZonedDateTime]
+    state: Option[String] = None,
+    description: Option[String] = None,
+    due_on: Option[ZonedDateTime] = None
 )

github4s/src/main/scala/github4s/domain/Project.scala

@@ -24,21 +24,20 @@ final case class Project(
     id: Long,
     node_id: String,
     name: String,
-    body: Option[String],
     number: Int,
     creator: Creator,
     created_at: String,
     updated_at: String,
-    organization_permission: Option[String],
-    `private`: Option[Boolean]
+    body: Option[String] = None,
+    organization_permission: Option[String] = None,
+    `private`: Option[Boolean] = None
 )
 
 final case class Creator(
     login: String,
     id: Long,
     node_id: String,
     avatar_url: String,
-    gravatar_id: Option[String],
     url: String,
     html_url: String,
     followers_url: String,
@@ -51,7 +50,8 @@ final case class Creator(
     events_url: String,
     received_events_url: String,
     `type`: String,
-    site_admin: Boolean
+    site_admin: Boolean,
+    gravatar_id: Option[String] = None
 )
 
 final case class Column(
@@ -70,11 +70,11 @@ final case class Card(
     project_url: String,
     id: Long,
     node_id: String,
-    note: Option[String],
     archived: Boolean,
     creator: Creator,
     created_at: String,
     updated_at: String,
     column_url: String,
-    content_url: Option[String]
+    note: Option[String] = None,
+    content_url: Option[String] = None
 )

github4s/src/main/scala/github4s/domain/PullRequest.scala

@@ -21,26 +21,26 @@ final case class PullRequest(
     number: Int,
     state: String,
     title: String,
-    body: Option[String],
     locked: Boolean,
     html_url: String,
     created_at: String,
-    updated_at: Option[String],
-    closed_at: Option[String],
-    merged_at: Option[String],
-    merge_commit_sha: Option[String],
-    base: Option[PullRequestBase],
-    head: Option[PullRequestBase],
-    user: Option[User],
-    assignee: Option[User]
+    body: Option[String] = None,
+    updated_at: Option[String] = None,
+    closed_at: Option[String] = None,
+    merged_at: Option[String] = None,
+    merge_commit_sha: Option[String] = None,
+    base: Option[PullRequestBase] = None,
+    head: Option[PullRequestBase] = None,
+    user: Option[User] = None,
+    assignee: Option[User] = None
 )
 
 final case class PullRequestBase(
-    label: Option[String],
     ref: String,
     sha: String,
-    user: Option[User],
-    repo: Option[Repository]
+    label: Option[String] = None,
+    user: Option[User] = None,
+    repo: Option[Repository] = None
 )
 
 final case class PullRequestFile(
@@ -53,8 +53,8 @@ final case class PullRequestFile(
     blob_url: String,
     raw_url: String,
     contents_url: String,
-    patch: Option[String],
-    previous_filename: Option[String]
+    patch: Option[String] = None,
+    previous_filename: Option[String] = None
 )
 sealed trait CreatePullRequest {
   def head: String
@@ -108,12 +108,12 @@ final case class NewPullRequestIssue(issue: Int)                 extends NewPull
 
 final case class PullRequestReview(
     id: Long,
-    user: Option[User],
     body: String,
     commit_id: String,
     state: PullRequestReviewState,
     html_url: String,
-    pull_request_url: String
+    pull_request_url: String,
+    user: Option[User] = None
 )
 
 sealed abstract class PullRequestReviewState(val value: String)