Skip to content

chore: Override the version of protobuf-bom to 4.28.3 in libraries-bom.#6836

Merged
blakeli0 merged 3 commits intomainfrom
upgrade-protobuf4
Oct 31, 2024
Merged

chore: Override the version of protobuf-bom to 4.28.3 in libraries-bom.#6836
blakeli0 merged 3 commits intomainfrom
upgrade-protobuf4

Conversation

@blakeli0
Copy link
Copy Markdown
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 review from a team, lqiu96, mpeddada1 and suztomo October 30, 2024 18:13
.append("- Google Cloud Core: ")
.append(versionlessCoordinatesToVersion.get("com.google.cloud:google-cloud-core"))
.append("\n");
report
Copy link
Copy Markdown
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.

Comment thread versions.txt Outdated
Comment thread release-note-generation/src/main/java/com/google/cloud/ReleaseNoteGeneration.java Outdated
Copy link
Copy Markdown
Member

@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
@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