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: Override the version of protobuf-bom to 4.28.3 in libraries-bom. #6836

Merged
merged 3 commits into from
Oct 31, 2024

Conversation

blakeli0
Copy link
Contributor

@blakeli0 blakeli0 commented Oct 29, 2024

In this PR:

  • Add a new protobuf-bom section to libraries-bom to override the protobuf runtime version to the latest.
  • Add a new generated release notes to direct customers to use libraries-bom-protobuf3 if there is any incompatibility issues.
  • Update the version of libraries-bom-protobuf3 to match the version of libraries-bom.
  • Disables protobuf-bom from being updated by renovate bot.

@product-auto-label product-auto-label bot added the size: s Pull request size is small. label Oct 29, 2024
@blakeli0 blakeli0 marked this pull request as ready for review October 30, 2024 18:13
@blakeli0 blakeli0 requested a review from a team as a code owner October 30, 2024 18:13
@@ -182,6 +182,9 @@ private void printKeyCoreLibraryDependencies(Bom bom) {
.append("- Google Cloud Core: ")
.append(versionlessCoordinatesToVersion.get("com.google.cloud:google-cloud-core"))
.append("\n");
report
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 was considering adding a unit test for it, but looks like we don't have any for this Core Library Dependencies section.

versions.txt Outdated Show resolved Hide resolved
Copy link
Contributor

@lqiu96 lqiu96 left a comment

Choose a reason for hiding this comment

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

LGTM. Added a nit for wording + tomo's comment

@blakeli0 blakeli0 merged commit 5662b26 into main Oct 31, 2024
28 of 29 checks passed
@blakeli0 blakeli0 deleted the upgrade-protobuf4 branch October 31, 2024 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: s Pull request size is small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants