Skip to content

feat(internal/librarian/java): generate module level parent and bom pom if missing#4983

Merged
zhumin8 merged 12 commits intogoogleapis:mainfrom
zhumin8:java-pom-library-bom-parent-1
Apr 1, 2026
Merged

feat(internal/librarian/java): generate module level parent and bom pom if missing#4983
zhumin8 merged 12 commits intogoogleapis:mainfrom
zhumin8:java-pom-library-bom-parent-1

Conversation

@zhumin8
Copy link
Copy Markdown
Contributor

@zhumin8 zhumin8 commented Apr 1, 2026

Support generate module level parent and bom pom.xml files from template if missing. Also fixed proto- and grpc- module groupId.

For #4922

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces the generation of BOM and Parent POM files for Java libraries. Key changes include refactoring the POM generation logic to support these new modules, updating data structures to include versioning, and adding helper functions for monorepo version lookup and group ID determination. A review comment suggests using the rootLibrary constant in an error message instead of a hardcoded string to improve maintainability.

@zhumin8 zhumin8 marked this pull request as ready for review April 1, 2026 15:41
@zhumin8 zhumin8 requested a review from a team as a code owner April 1, 2026 15:41
@zhumin8 zhumin8 requested a review from JoeWang1127 April 1, 2026 20:50
Copy link
Copy Markdown
Contributor

@JoeWang1127 JoeWang1127 left a comment

Choose a reason for hiding this comment

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

Left some comments on the tests and doc, feel free to refactor in a follow up.

@zhumin8 zhumin8 enabled auto-merge (squash) April 1, 2026 21:17
@zhumin8 zhumin8 merged commit 9619a20 into googleapis:main Apr 1, 2026
17 checks passed
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