Skip to content

Conversation

@JoeWang1127
Copy link
Contributor

@JoeWang1127 JoeWang1127 commented Apr 30, 2024

In this PR:

  • Update renovate config to only update libraries bom version in README.md in 4 modules (java-dns, java-grafeas, java-notification and java-vertexai) since this value in other modules is updated by hermetic build scripts.

To verify the change:

export LOG_LEVEL=debug && npx --yes --package renovate -- renovate --platform=local

...
DEBUG: Matched 4 file(s) for manager regex: java-dns/README.md, java-grafeas/README.md, java-notification/README.md, java-vertexai/README.md (repository=local)
...

@JoeWang1127
Copy link
Contributor Author

"generation diff / root-pom" check will pass when we update the library generation image tag.

@JoeWang1127 JoeWang1127 marked this pull request as ready for review April 30, 2024 21:14
@JoeWang1127 JoeWang1127 requested a review from a team as a code owner April 30, 2024 21:14
"depNameTemplate": "com.google.cloud:sdk-platform-java-config",
"datasourceTemplate": "maven"
},
{
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we want to remove this section all together, we still need it for vertex-ai, dns, notification and grafeas.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I changed the config to only update libraries bom version in these 4 modules.

PR title and description are also modified.

@JoeWang1127 JoeWang1127 changed the title chore: update renovate to exclude libraries-bom update in README.md chore: update renovate to only update libraries-bom version in README.md in 4 modules Apr 30, 2024
@JoeWang1127 JoeWang1127 requested a review from blakeli0 April 30, 2024 22:21
@JoeWang1127 JoeWang1127 merged commit 6388fbc into main May 1, 2024
@JoeWang1127 JoeWang1127 deleted the chore/update-renovate branch May 1, 2024 17:19
burkedavison added a commit that referenced this pull request Dec 26, 2024
Original PR to limit renovate-bot's libraries-bom updates due to conflict with hermetic build: #10768

However, we want the native-image-sample to be included in these updates. It is not updated by hermetic build.
burkedavison added a commit that referenced this pull request Dec 26, 2024
…on (#11367)

Original PR to limit renovate-bot's libraries-bom updates due to conflict with hermetic build: #10768

However, we want the native-image-sample to be included in these updates. It is not updated by hermetic build.
lqiu96 pushed a commit that referenced this pull request Jul 7, 2025
…on (#11367)

Original PR to limit renovate-bot's libraries-bom updates due to conflict with hermetic build: #10768

However, we want the native-image-sample to be included in these updates. It is not updated by hermetic build.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants