Skip to content

chore: add markers in pom.xml#13353

Closed
zhumin8 wants to merge 4 commits into
mainfrom
add-pom-markers-0603
Closed

chore: add markers in pom.xml#13353
zhumin8 wants to merge 4 commits into
mainfrom
add-pom-markers-0603

Conversation

@zhumin8

@zhumin8 zhumin8 commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Add comment markers in pom.xml for generated proto and grpc artifacts.
These markers are added in parent, BOM and client pom.xml files for each library.

Also included in this PR is a one line fix for sdk-platform-java/hermetic_build/library_generation/owlbot/src/fix_poms.py, so that it ignores comments during extra module detection, and does not produce broken pom.xml files when the original has comments.
The consequence of this minimal fix is that comments added will be wiped out when regenerated (see 557b1ed). This is acceptable as is only during migration and will re-run and add markers as needed.

For googleapis/librarian#6134

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

Copy link
Copy Markdown
Contributor

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 adds generator marker comments (such as {x-generated-dependencies-start}, {x-generated-proto-dependencies-start}, {x-generated-grpc-dependencies-start}, and {x-generated-modules-start}) to the pom.xml files across numerous Google Cloud Java client libraries. These markers delineate sections of generated dependencies and modules, likely to assist automated tools in updating them. There are no review comments on this pull request, and I have no feedback to provide.

@sonarqubecloud

sonarqubecloud Bot commented Jun 3, 2026

Copy link
Copy Markdown

@sonarqubecloud

sonarqubecloud Bot commented Jun 3, 2026

Copy link
Copy Markdown

@zhumin8

zhumin8 commented Jun 3, 2026

Copy link
Copy Markdown
Contributor Author

hermetic build generation workflow failed on backstory, is expected.
This PR is before #13351

@zhumin8 zhumin8 marked this pull request as ready for review June 3, 2026 19:42
@zhumin8 zhumin8 requested review from a team as code owners June 3, 2026 19:42
@zhumin8 zhumin8 requested review from JoeWang1127 and sofisl June 3, 2026 19:42
@zhumin8

zhumin8 commented Jun 4, 2026

Copy link
Copy Markdown
Contributor Author

close as I don't want to manually resolve conflict. And will raise again when ready to turn done hermetic build.

@zhumin8 zhumin8 closed this Jun 4, 2026
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