Skip to content

build: centralize quick-build profile in java-shared-config - #14106

Open
lqiu96 wants to merge 1 commit into
mainfrom
centralize-quick-build-profile
Open

build: centralize quick-build profile in java-shared-config#14106
lqiu96 wants to merge 1 commit into
mainfrom
centralize-quick-build-profile

Conversation

@lqiu96

@lqiu96 lqiu96 commented Aug 18, 2026

Copy link
Copy Markdown
Member

Move the quick-build profile to shared-configs so that it can be used for all modules in the monorepo. Should be properly inherited by all the modules as it's the top level config for everything in the monorepo.

@lqiu96
lqiu96 requested review from a team as code owners August 18, 2026 17:27

@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 centralizes the quick-build profile configuration into java-shared-config/java-shared-config/pom.xml and removes its redundant definitions from other parent POMs. It also introduces a custom shade.skip property to toggle the Maven shade plugin. The reviewer suggests using the standard, natively supported maven.shade.skip property instead of the custom shade.skip property to simplify configuration.

Comment thread java-shared-config/java-shared-config/pom.xml
Comment thread java-shared-config/java-shared-config/pom.xml
@sonarqubecloud

Copy link
Copy Markdown

@sonarqubecloud

Copy link
Copy Markdown

@lqiu96 lqiu96 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 18, 2026
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 18, 2026
@lqiu96 lqiu96 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 18, 2026
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 18, 2026
@lqiu96 lqiu96 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 18, 2026
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 18, 2026
@lqiu96 lqiu96 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 18, 2026
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 18, 2026
@lqiu96
lqiu96 enabled auto-merge (squash) August 18, 2026 22:30
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