Skip to content
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

chore: update renovate to only update libraries-bom version in README.md in 4 modules #10768

Merged
merged 3 commits into from
May 1, 2024

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
@@ -41,16 +41,6 @@
"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
29 of 30 checks passed
@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.
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