-
Notifications
You must be signed in to change notification settings - Fork 625
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
[ISSUE #4845] Apply dependency management to grpcVersion
and protobufVersion
.
#4988
base: master
Are you sure you want to change the base?
Conversation
grpcVersion
and protobufVersion
.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4988 +/- ##
=========================================
Coverage 15.64% 15.64%
Complexity 1777 1777
=========================================
Files 1001 1001
Lines 33777 33777
Branches 2917 2917
=========================================
Hits 5284 5284
Misses 27975 27975
Partials 518 518 ☔ View full report in Codecov by Sentry. |
build.gradle
Outdated
def opentelemetryVersion = '1.36.0' | ||
def cloudeventsVersion = '3.0.0' | ||
def cloudeventsVersion = '2.5.0' | ||
def curatorVersion = '5.7.0' |
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.
Please do not downgrade.
build.gradle
Outdated
dependency "org.springframework.boot:spring-boot-starter-jetty:2.7.18" | ||
|
||
dependency "com.google.protobuf:protobuf-java-util:${protobufVersion}" | ||
dependency "com.google.protobuf:protobuf-java:${protobufVersion}" | ||
dependency "com.google.protobuf:protoc:${protocVersion}" | ||
|
||
dependency "com.mysql:mysql-connector-j:8.4.0" |
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.
The google.protobuf dependencies should be placed after the grpc dependencies. The same as the protobufVersion
var.
def protobufVersion = '3.25.3' | ||
def protocVersion = protobufVersion | ||
|
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.
Redundant protobufVersion
field.
Fixes #4845
Previous PR: #4876
Motivation
update grpc version and protobuf version
Modifications
Describe the modifications you've done.
Documentation