Remove TODOs

commercetools · Sep 24, 2024 · ac4ba70 · ac4ba70
1 parent 7a9c0ec
commit ac4ba70
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/aws/sqs/src/main/scala/com/commercetools/queue/aws/sqs/SQSMessageBatch.scala b/aws/sqs/src/main/scala/com/commercetools/queue/aws/sqs/SQSMessageBatch.scala
@@ -50,7 +50,6 @@ private class SQSMessageBatch[F[_], T](
         )
       }
     }.void
-  // TODO adapt error
 
   override def nackAll: F[Unit] = F.fromCompletableFuture {
     F.delay {
@@ -72,5 +71,5 @@ private class SQSMessageBatch[F[_], T](
         req
       )
     }
-  }.void // TODO adapt error
+  }.void
 }
diff --git a/gcp/pubsub/integration/src/test/scala/com/commercetools/queue/pubsub/PubSubClientSuite.scala b/gcp/pubsub/integration/src/test/scala/com/commercetools/queue/pubsub/PubSubClientSuite.scala
@@ -38,7 +38,7 @@ class PubSubClientSuite extends QueueClientSuite {
       ifTrue = IO.pure(("test-project", NoCredentialsProvider.create(), Some("localhost:8042"))),
       ifFalse = for {
         project <- string("GCP_PUBSUB_PROJECT")
-        credentials = NoCredentialsProvider.create() // TODO
+        credentials = NoCredentialsProvider.create()
       } yield (project, credentials, None)
     )
 

diff --git a/gcp/pubsub/src/main/scala/com/commercetools/queue/gcp/pubsub/PubSubMessageBatch.scala b/gcp/pubsub/src/main/scala/com/commercetools/queue/gcp/pubsub/PubSubMessageBatch.scala
@@ -41,7 +41,7 @@ private class PubSubMessageBatch[F[_], T](
               .newBuilder()
               .setSubscription(subscriptionName.toString)
               .addAllAckIds(payload.map(_.underlying.getAckId).asJava)
-              .build()))).void // TODO mladens adapt error
+              .build()))).void
 
   override def nackAll: F[Unit] = wrapFuture(
     F.delay(
@@ -53,5 +53,5 @@ private class PubSubMessageBatch[F[_], T](
             .setSubscription(subscriptionName.toString)
             .setAckDeadlineSeconds(0)
             .addAllAckIds(payload.map(_.underlying.getAckId).asJava)
-            .build()))).void // TODO mladens adapt error
+            .build()))).void
 }