-
Notifications
You must be signed in to change notification settings - Fork 17
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 upstream agent #49
Conversation
@@ -15,3 +17,10 @@ updates: | |||
github-actions: | |||
patterns: | |||
- "*" | |||
|
|||
# Gradle | |||
- package-ecosystem: "gradle" |
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.
[for reviewer] this configures dependabot to attempt updating gradle dependencies in gradle/libs.versions.toml
, maybe other files are covered as well (but this will be the central place where we attempt to configure dependencies in the future).
|
||
# Gradle | ||
- package-ecosystem: "gradle" | ||
directory: "/gradle/" # for now only on the version catalog to prevent noise |
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.
[for reviewer] for now we only try to update versions set in "version catalog", covering all dependencies should be part of #8
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.
Very nice!
@@ -16,8 +16,8 @@ subprojects { | |||
ext { | |||
deps = [ | |||
autoservice: [ | |||
"com.google.auto.service:auto-service:" + libraries.versions.autoservice.get(), | |||
"com.google.auto.service:auto-service-annotations:" + libraries.versions.autoservice.get(), | |||
"com.google.auto.service:auto-service:" + catalog.versions.autoservice.get(), |
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.
I guess we can't use libraries
instead of versions
here?
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.
I don't know yet for those and plan to tackle it with #8 later to focus on the otel parts.
|
||
testImplementation("org.mockito:mockito-core:5.5.0") | ||
|
||
testImplementation(enforcedPlatform("org.junit:junit-bom:" + libraries.versions.junit.get().toString())) | ||
testImplementation(enforcedPlatform("org.junit:junit-bom:" + catalog.versions.junit.get().toString())) |
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.
Same here: Impossible to use a library
instead of version, correct?
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.
No, here it's possible to use the catalog in a declarative way, but I will do that when dealing with #8 to keep changes minimal here.
Closes #15