-
Notifications
You must be signed in to change notification settings - Fork 100
WIP - feat: configurable crypto provider #4182
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
| "dep:tokio", | ||
| "dep:tracing", | ||
| ] | ||
| _internal-grpc-client = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are dep:google-cloud-auth(40 line and 57 line) and dep:gax( 43 line and 60 line ) duplicate imports?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dep:gax duplicate imports
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #4182 +/- ##
=======================================
Coverage 94.77% 94.77%
=======================================
Files 187 187
Lines 7119 7119
=======================================
Hits 6747 6747
Misses 372 372 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
435f5e7 to
e65c508
Compare
|
It is recommended to prioritize the three crates of 'google-cloud-storage','google-cloud-auth', and 'google-cloud-gax-internal' and release new versions. |
Fix formatting Add requirements for gax-internal
e65c508 to
9fa5fc2
Compare
|
Closing for now, track the work in #4170 |
This is just for testing, I need to break down the PR into smaller changes, complete the changes for generated clients, and add some builds to run the new
crypto-providerstests.