codxbrexx commented on code in PR #7038:
URL: https://github.com/apache/opendal/pull/7038#discussion_r2630515111


##########
core/services/surrealdb/Cargo.toml:
##########
@@ -33,6 +33,12 @@ all-features = true
 [dependencies]
 opendal-core = { path = "../../core", version = "0.55.0", default-features = 
false }
 
+# Pin async-graphql components to avoid breaking changes in v7.0.16+
+# The derive macro in 7.0.17 is incompatible with earlier versions
+async-graphql = "=7.0.11"
+async-graphql-derive = "=7.0.11"
+async-graphql-parser = "=7.0.11"
+async-graphql-value = "=7.0.11"

Review Comment:
   Thanks for catching that. I added those pins to work around a local build 
breakage I was seeing, but I agree we should avoid pinning indirect 
dependencies explicitly. I've reverted the changes to 
   surrealdb/Cargo.toml
    and regenerated Cargo.lock minimally.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to