Change warnings to todos

apollographql · Jan 5, 2024 · 2f8b56e · 2f8b56e
1 parent a4ecf54
commit 2f8b56e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Tests/ApolloTests/GraphQLResultTests.swift b/Tests/ApolloTests/GraphQLResultTests.swift
@@ -116,6 +116,6 @@ final class GraphQLResultTests: XCTestCase {
 
   // MARK: Incremental merging tests
 
-  #warning("Need incremental merging tests here!")
+  // TODO: Need more incremental merging tests here
 
 }
diff --git a/apollo-ios/Sources/Apollo/GraphQLResult.swift b/apollo-ios/Sources/Apollo/GraphQLResult.swift
@@ -151,7 +151,7 @@ fileprivate extension DataDict {
     }
 
     let mergedData = try pathDataDict._data.merging(newDataDict._data) { current, new in
-      #warning("This is a hack fix for __typename being returned with fragments, need a proper solution!")
+      // TODO: This is a quick fix for __typename being returned with fragments, needs a rethink!
       if current != new {
         throw Error.cannotOverwriteData(current, new)
       }