Skip to content

ci: quick-build profile skip shading #14046

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.

Sign up for GitHub

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

Open
lqiu96 wants to merge 8 commits into
base: main
Choose a base branch
from
Open

ci: quick-build profile skip shading #14046

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
fdcb8d0
build: skip maven-shade-plugin execution in quick-build profile
lqiu96 Aug 11, 2026
1eed6a3
build: standardize on shade.skip property for quick-build profile
lqiu96 Aug 11, 2026
03cabcf
build: rely on native shade.skip parameter injection without redundan…
lqiu96 Aug 11, 2026
963a2f8
build: skip maven-shade-plugin in quick-build profile
lqiu96 Aug 11, 2026
2f79b14
build: explicitly map shade.skip in shaded module configurations
lqiu96 Aug 11, 2026
2b8f7a0
build: default shade.skip to false with explanatory comments
lqiu96 Aug 11, 2026
b2039d6
build: map shade.skip across all modules using maven-shade-plugin
lqiu96 Aug 11, 2026
8863b3c
build: revert skip-shading for gapic-generator-java
lqiu96 Aug 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions google-cloud-pom-parent/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,9 @@
<properties>
<skipUnitTests>false</skipUnitTests>
<checkstyle.header.file>java.header</checkstyle.header.file>
<!-- maven-shade-plugin does not define a built-in user property for its skip parameter;
shade.skip is declared here and mapped in shaded modules to allow quick-build to toggle it off. -->
<shade.skip>false</shade.skip>
</properties>

<developers>
Expand Down Expand Up @@ -72,6 +75,7 @@
<fmt.skip>true</fmt.skip>
<flatten.skip>true</flatten.skip>
<mdep.analyze.skip>true</mdep.analyze.skip>
<shade.skip>true</shade.skip>
Comment thread
lqiu96 marked this conversation as resolved.
</properties>
</profile>
<profile>
Expand Down
1 change: 1 addition & 0 deletions java-bigquery/benchmark/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,7 @@
<goal>shade</goal>
</goals>
<configuration>
<skip>${shade.skip}</skip>
<finalName>${uberjar.name}</finalName>
<transformers>
<transformer implementation="org.apache.maven.plugins.shade.resource.ManifestResourceTransformer">
Expand Down
3 changes: 3 additions & 0 deletions java-bigtable/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -286,6 +286,9 @@
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-shade-plugin</artifactId>
<version>3.6.0</version>
<configuration>
<skip>${shade.skip}</skip>
</configuration>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
Expand Down
1 change: 1 addition & 0 deletions java-bigtable/test-proxy/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -158,6 +158,7 @@
<goal>shade</goal>
</goals>
<configuration>
<skip>${shade.skip}</skip>
<filters>
<filter>
<artifact>*:*</artifact>
Expand Down
4 changes: 4 additions & 0 deletions java-cloud-bom/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,9 @@

<properties>
<enforcer.skip>true</enforcer.skip>
<!-- maven-shade-plugin does not define a built-in user property for its skip parameter;
shade.skip is declared here and mapped in shaded modules to allow quick-build to toggle it off. -->
<shade.skip>false</shade.skip>
</properties>

<modules>
Expand Down Expand Up @@ -116,6 +119,7 @@
<fmt.skip>true</fmt.skip>
<flatten.skip>true</flatten.skip>
<mdep.analyze.skip>true</mdep.analyze.skip>
<shade.skip>true</shade.skip>
Comment thread
lqiu96 marked this conversation as resolved.
</properties>
</profile>
<profile>
Expand Down
1 change: 1 addition & 0 deletions java-spanner-jdbc/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -275,6 +275,7 @@
<goal>shade</goal>
</goals>
<configuration>
<skip>${shade.skip}</skip>
<createSourcesJar>true</createSourcesJar>
<shadeSourcesContent>true</shadeSourcesContent>
<shadedArtifactAttached>true</shadedArtifactAttached>
Expand Down
1 change: 1 addition & 0 deletions java-storage-nio/google-cloud-nio/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -129,6 +129,7 @@
</dependency>
</dependencies>
<configuration>
<skip>${shade.skip}</skip>
<shadedArtifactAttached>true</shadedArtifactAttached>
<relocations>
<relocation>
Expand Down
1 change: 1 addition & 0 deletions java-storage/storage-shared-benchmarking/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,7 @@
<goal>shade</goal>
</goals>
<configuration>
<skip>${shade.skip}</skip>
<finalName>${uberjar.name}</finalName>
<transformers>
<transformer implementation="org.apache.maven.plugins.shade.resource.ManifestResourceTransformer">
Expand Down
Loading