Skip to content

feat(gax): implement grpc-rust client constructors#6046

Merged
joshuatants merged 1 commit into
googleapis:grpc_rustfrom
joshuatants:grpc_rust_prototype_construction
Jul 16, 2026
Merged

feat(gax): implement grpc-rust client constructors#6046
joshuatants merged 1 commit into
googleapis:grpc_rustfrom
joshuatants:grpc_rust_prototype_construction

Conversation

@joshuatants

@joshuatants joshuatants commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

for #5991

Comment thread src/gax-internal/src/grpc/grpc_rust.rs Outdated
Comment thread src/gax-internal/src/grpc/grpc_rust.rs
Comment thread src/gax-internal/src/grpc/grpc_rust.rs Outdated
Comment thread src/gax-internal/src/grpc/grpc_rust.rs Outdated
@joshuatants
joshuatants force-pushed the grpc_rust_prototype_construction branch 2 times, most recently from 7a3d764 to 839c0d2 Compare July 15, 2026 06:49
@joshuatants
joshuatants force-pushed the grpc_rust_prototype_construction branch from 839c0d2 to af40da2 Compare July 15, 2026 06:56
@joshuatants

Copy link
Copy Markdown
Contributor Author

@dfawley I cleaned up the comments after our discussion this morning. Could you take a look at the rest of the code? Thank you!

@joshuatants
joshuatants marked this pull request as ready for review July 15, 2026 08:20
@joshuatants
joshuatants requested review from a team as code owners July 15, 2026 08:20
@joshuatants
joshuatants requested a review from coryan July 15, 2026 08:21
@joshuatants

Copy link
Copy Markdown
Contributor Author

@dfawley I cleaned up the comments after our discussion this morning. Could you take a look at the rest of the code? Thank you!

@dfawley merging to grpc_rust first so that I can continue working on the bidi implementation. Happy to address any post-merge comments.

@joshuatants
joshuatants merged commit aa43376 into googleapis:grpc_rust Jul 16, 2026
2 checks passed
@joshuatants
joshuatants deleted the grpc_rust_prototype_construction branch July 16, 2026 02:17
joshuatants added a commit to joshuatants/google-cloud-rust that referenced this pull request Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants