vaticle / dependencies

Bazel dependency declarations for build tools reused across @vaticle repositories (only for @vaticle)
Mozilla Public License 2.0
12 stars 23 forks source link

Disable non-camel-case warnings in tonic compile #485

Closed dmitrii-ubskii closed 8 months ago

dmitrii-ubskii commented 8 months ago

What is the goal of this PR?

tonic-build gives gRPC methods that return a stream (e.g. transaction()) a dedicated type alias for the return type. The name of the type is {method name}Stream, which violates Rust naming conventions. We don't have control over those type names though, so the next best thing is disabling the diagnostic altogether.

typedb-bot commented 8 months ago

PR Review Checklist

Do not edit the content of this comment. The PR reviewer should simply update this comment by ticking each review item below, as they get completed.


Trivial Change

Code

Architecture