Change updatePullRequest to return Unit

Author: alejandrohdezma

modules/core/src/main/scala/org/scalasteward/core/forge/ForgeApiAlg.scala

@@ -27,11 +27,7 @@ trait ForgeApiAlg[F[_]] {
 
   def createPullRequest(repo: Repo, data: NewPullRequestData): F[PullRequestOut]
 
-  def updatePullRequest(
-      number: PullRequestNumber,
-      repo: Repo,
-      data: NewPullRequestData
-  ): F[PullRequestOut]
+  def updatePullRequest(number: PullRequestNumber, repo: Repo, data: NewPullRequestData): F[Unit]
 
   def closePullRequest(repo: Repo, number: PullRequestNumber): F[PullRequestOut]
 

modules/core/src/main/scala/org/scalasteward/core/forge/azurerepos/AzureReposApiAlg.scala

@@ -58,7 +58,7 @@ final class AzureReposApiAlg[F[_]](
       number: PullRequestNumber,
       repo: Repo,
       data: NewPullRequestData
-  ): F[PullRequestOut] =
+  ): F[Unit] =
     F.raiseError(new NotImplementedError(s"updatePullRequest($number, $repo, $data)"))
 
   // https://docs.microsoft.com/en-us/rest/api/azure/devops/git/pull-requests/update?view=azure-devops-rest-7.1

modules/core/src/main/scala/org/scalasteward/core/forge/bitbucket/BitbucketApiAlg.scala

@@ -101,7 +101,7 @@ class BitbucketApiAlg[F[_]](
       number: PullRequestNumber,
       repo: Repo,
       data: NewPullRequestData
-  ): F[PullRequestOut] =
+  ): F[Unit] =
     F.raiseError(new NotImplementedError(s"updatePullRequest($number, $repo, $data)"))
 
   override def getBranch(repo: Repo, branch: Branch): F[BranchOut] =

modules/core/src/main/scala/org/scalasteward/core/forge/bitbucketserver/BitbucketServerApiAlg.scala

@@ -90,7 +90,7 @@ final class BitbucketServerApiAlg[F[_]](
       number: PullRequestNumber,
       repo: Repo,
       data: NewPullRequestData
-  ): F[PullRequestOut] =
+  ): F[Unit] =
     F.raiseError(new NotImplementedError(s"updatePullRequest($number, $repo, $data)"))
 
   private def useDefaultReviewers(repo: Repo): F[List[Reviewer]] =

modules/core/src/main/scala/org/scalasteward/core/forge/gitea/GiteaApiAlg.scala

@@ -190,7 +190,7 @@ final class GiteaApiAlg[F[_]: HttpJsonClient](
       number: PullRequestNumber,
       repo: Repo,
       data: NewPullRequestData
-  ): F[PullRequestOut] =
+  ): F[Unit] =
     F.raiseError(new NotImplementedError(s"updatePullRequest($number, $repo, $data)"))
 
   override def closePullRequest(repo: Repo, number: PullRequestNumber): F[PullRequestOut] = {

modules/core/src/main/scala/org/scalasteward/core/forge/github/GitHubApiAlg.scala

@@ -70,7 +70,7 @@ final class GitHubApiAlg[F[_]](
       number: PullRequestNumber,
       repo: Repo,
       data: NewPullRequestData
-  ): F[PullRequestOut] = {
+  ): F[Unit] = {
     val payload = UpdatePullRequestPayload.from(data)
 
     val update = client
@@ -82,11 +82,11 @@ final class GitHubApiAlg[F[_]](
       .adaptErr(SecondaryRateLimitExceeded.fromThrowable)
 
     for {
-      pullRequestOut <- update
+      _ <- update
       _ <- F.whenA(data.labels.nonEmpty)(labelPullRequest(repo, number, data.labels))
       _ <- F.whenA(data.assignees.nonEmpty)(addAssignees(repo, number, data.assignees))
       _ <- F.whenA(data.reviewers.nonEmpty)(addReviewers(repo, number, data.reviewers))
-    } yield pullRequestOut
+    } yield ()
   }
 
   /** https://docs.github.com/en/rest/repos/branches?apiVersion=2022-11-28#get-branch */

modules/core/src/main/scala/org/scalasteward/core/forge/gitlab/GitLabApiAlg.scala

@@ -252,7 +252,7 @@ final class GitLabApiAlg[F[_]: Parallel](
       number: PullRequestNumber,
       repo: Repo,
       data: NewPullRequestData
-  ): F[PullRequestOut] =
+  ): F[Unit] =
     F.raiseError(new NotImplementedError(s"updatePullRequest($number, $repo, $data)"))
 
   private def mergePipelineUponSuccess(repo: Repo, mr: MergeRequestOut): F[MergeRequestOut] =

modules/core/src/test/scala/org/scalasteward/core/forge/github/GitHubApiAlgTest.scala

@@ -337,14 +337,7 @@ class GitHubApiAlgTest extends CatsEffectSuite with Http4sDsl[MockEff] {
 
     val pr = gitHubApiAlg.updatePullRequest(number, repo, data).runA(state)
 
-    val expected = PullRequestOut(
-      uri"https://github.com/octocat/Hello-World/pull/42",
-      PullRequestState.Open,
-      number,
-      "updated-title"
-    )
-
-    assertIO(pr, expected)
+    assertIO(pr, ())
   }
 
   test("createPullRequest with assignees and reviewers") {
@@ -376,14 +369,7 @@ class GitHubApiAlgTest extends CatsEffectSuite with Http4sDsl[MockEff] {
 
     val pr = gitHubApiAlg.updatePullRequest(number, repo, data).runA(state)
 
-    val expected = PullRequestOut(
-      uri"https://github.com/octocat/Hello-World/pull/42",
-      PullRequestState.Open,
-      number,
-      "updated-title"
-    )
-
-    assertIO(pr, expected)
+    assertIO(pr, ())
   }
 
   test("createPullRequest with assignees and reviewers should not fail if can't assign") {
@@ -427,14 +413,7 @@ class GitHubApiAlgTest extends CatsEffectSuite with Http4sDsl[MockEff] {
         .updatePullRequest(number, repo.copy(repo = "cant-assign-reviewers"), data)
         .runA(state)
 
-    val expected = PullRequestOut(
-      uri"https://github.com/fthomas/cant-assign-reviewers/pull/42",
-      PullRequestState.Open,
-      number,
-      "updated-title"
-    )
-
-    assertIO(pr, expected)
+    assertIO(pr, ())
   }
 
   test("createPullRequest with labels") {
@@ -466,14 +445,7 @@ class GitHubApiAlgTest extends CatsEffectSuite with Http4sDsl[MockEff] {
 
     val pr = gitHubApiAlg.updatePullRequest(number, repo, data).runA(state)
 
-    val expected = PullRequestOut(
-      uri"https://github.com/octocat/Hello-World/pull/42",
-      PullRequestState.Open,
-      number,
-      "updated-title"
-    )
-
-    assertIO(pr, expected)
+    assertIO(pr, ())
   }
 
   test("createPullRequest should fail when can't add labels") {