-
Notifications
You must be signed in to change notification settings - Fork 834
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
Update transitive dependencies #1003
Update transitive dependencies #1003
Conversation
/uncc @jklaise |
/cc @cliveseldon @gsunner |
Looks solid! Did all the e2e tests run successfully? |
@axsaucedo Yuup, e2e tests seem to pass! |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: axsaucedo The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Fixes #1002.
Changelog
checker-qual
tov2.11.1
. There is an open PR to change it inguava
here Upgrade checker-qual to v2.11.1 google/guava#3671.proto-google-common-protos
tov1.16.0
. There is an open PR to change it ingrpc-protobuf
here Update proto-google-common-proto to v1.17.0 grpc/grpc-java#6336.okhttp
tov4.2.2
. There is an open issue to change it injaeger-client
here Update to okhttp v4.2.2 jaegertracing/jaeger-client-java#666.Notes
The latest version of
proto-google-common-protos
seems to bev1.17.0
according to Maven Repository. However, as it's pointed out in this issue grpc/grpc-java#6336, the https://github.com/googleapis/common-protos-java repo seems to be still pointing tov1.16.0
. Therefore, it seems sensible to update tov1.16.0
until it's confirmed which one is the latest version.