-
Notifications
You must be signed in to change notification settings - Fork 67
chore: Add renovate custom manager to update upper-bound files #3894
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
|
|
5e8d7b3 to
ba9bb67
Compare
| <properties> | ||
| <skipUnitTests>false</skipUnitTests> | ||
| <checkstyle.header.file>java.header</checkstyle.header.file> | ||
| <maven.compiler.release>8</maven.compiler.release> |
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 think changes in this file is unintentional? If yes, can we revert it?
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.
Moved the non-dependency property up (I think above is moreso a section for maven plugins.
| javax.annotation:javax.annotation-api,javax.annotation-api=1.3.2 | ||
| io.grpc:grpc-bom,grpc=1.74.0 | ||
| com.google.auth:google-auth-library-bom,google.auth=1.37.1 | ||
| com.google.http-client:google-http-client,google.http-client=1.47.1 | ||
| com.google.code.gson:gson,gson=2.13.1 |
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.
These may have some old versions that weren't merged properly. I think we should see renovate-bot raise a PR for these once this is merged in.
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.
Where are the old versions pulled from? Can we keep the latest versions?
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.
These are just old versions from this stale PR. Hopefully renovate bot can group the deps in these files in one deps PR.
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.
SG. As long as we upgrade them before releasing sdk-platform-java, we should be good.
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.
SG. As long as we upgrade them before releasing sdk-platform-java, we should be good.
I will merge this in and test this. Hopefully this should all come in as one big PR!
|
|



Dependencies CI Test: https://github.com/googleapis/sdk-platform-java/actions/runs/18327368470/job/52195017520
Changes
In order to keep the current Maven pom.xml files the same, we create a new format in the dependencies.txt file:
{GroupID}:{ArtifactID},{MavenPropertyID}={Version}. TheGroupID,ArtifactID, andVersionvalues are by renovate bot to determine there should be a dependency bump. TheMavenPropertyIDandVersionvalues are used to build the maven test command.The format may not look intuitive and can be changed in the future. Values/ Separators were chosen so that it doesn't have any weird edge cases/ impact in shell scripts (avoiding ';', '*', '|' characters)
Local Renovate Bot Invocation
Local invocation of renovate shows that it is able to match the versions: