333: Resolve swiftlint warnings

Signed-off-by: Eddie Seay <[email protected]>
connectrpc · Feb 5, 2025 · b6b2ddd · b6b2ddd
1 parent ccfb544
commit b6b2ddd
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -123,7 +123,7 @@ jobs:
   run-swiftlint:
     runs-on: ubuntu-latest
     container:
-      image: ghcr.io/realm/swiftlint:0.57.1
+      image: ghcr.io/realm/swiftlint:0.58.2
     steps:
       - uses: actions/checkout@v4
       - name: Run SwiftLint

diff --git a/Libraries/Connect/Internal/Unary/UnaryAsyncWrapper.swift b/Libraries/Connect/Internal/Unary/UnaryAsyncWrapper.swift
@@ -21,7 +21,7 @@ import SwiftProtobuf
 /// https://forums.swift.org/t/how-to-use-withtaskcancellationhandler-properly/54341/37
 /// https://stackoverflow.com/q/71898080
 @available(iOS 13, *)
-actor UnaryAsyncWrapper<Output: ProtobufMessage>: Sendable {
+actor UnaryAsyncWrapper<Output: ProtobufMessage> {
     private var cancelable: Cancelable?
     private let sendUnary: PerformClosure
 
@@ -44,7 +44,12 @@ actor UnaryAsyncWrapper<Output: ProtobufMessage>: Sendable {
         await withTaskCancellationHandler {
             await withCheckedContinuation { continuation in
                 guard !Task.isCancelled else {
-                    continuation.resume(returning: .init(code: .canceled, result: .failure(.canceled())))
+                    continuation.resume(
+                        returning: ResponseMessage(
+                            code: .canceled,
+                            result: .failure(.canceled())
+                        )
+                    )
                     return
                 }
 
@@ -58,8 +63,8 @@ actor UnaryAsyncWrapper<Output: ProtobufMessage>: Sendable {
                 }
             }
         } onCancel: {
-            // When `Task.cancel` signals for this function to be canceled, the underlying function will be canceled
-            // as well.
+            // When `Task.cancel` signals for this function to be canceled,
+            // the underlying function will be canceled as well.
             Task(priority: .high) {
                 await self.cancelable?.cancel()
             }