chore(java-cloud-bom): fix lint failure by adding missing modules to reactor and skipping deploy/checkstyle

Author: jinseopkim0

java-cloud-bom/dashboard/pom.xml

@@ -31,6 +31,8 @@
     <maven.compiler.source>1.8</maven.compiler.source>
     <maven.compiler.target>1.8</maven.compiler.target>
 
+    <maven.deploy.skip>true</maven.deploy.skip>
+    <checkstyle.skip>true</checkstyle.skip>
     <guava.version>33.6.0-jre</guava.version>
   </properties>
 

java-cloud-bom/dashboard/src/main/java/com/google/cloud/tools/opensource/cloudbomdashboard/ArtifactResults.java

@@ -41,7 +41,9 @@ void addResult(String testName, int failures) {
     results.put(testName, failures);
   }
 
-  /** @return true for pass, false for fail, null for unknown test */
+  /**
+   * @return true for pass, false for fail, null for unknown test
+   */
   @Nullable
   public Boolean getResult(String testName) {
     Integer failures = results.get(testName);
@@ -55,13 +57,17 @@ public String getCoordinates() {
     return Artifacts.toCoordinates(artifact);
   }
 
-  /** @return message of exception occurred when running test, null for no exception */
+  /**
+   * @return message of exception occurred when running test, null for no exception
+   */
   @Nullable
   public String getExceptionMessage() {
     return exceptionMessage;
   }
 
-  /** @return number of times the specified test failed. Returns 0 if the test was not run. */
+  /**
+   * @return number of times the specified test failed. Returns 0 if the test was not run.
+   */
   public int getFailureCount(String testName) {
     Integer failures = results.get(testName);
     if (failures == null) {

java-cloud-bom/dashboard/src/main/java/com/google/cloud/tools/opensource/cloudbomdashboard/DashboardMain.java

@@ -88,8 +88,12 @@ public class DashboardMain {
    * snapshot version.
    */
   public static void main(String[] arguments)
-      throws IOException, TemplateException, RepositoryException, URISyntaxException,
-          ParseException, MavenRepositoryException {
+      throws IOException,
+          TemplateException,
+          RepositoryException,
+          URISyntaxException,
+          ParseException,
+          MavenRepositoryException {
     DashboardArguments dashboardArguments = DashboardArguments.readCommandLine(arguments);
 
     // If looking to edit the dashboard structure, see DashboardMain#generateDashboard.
@@ -130,7 +134,10 @@ public static void main(String[] arguments)
   }
 
   private static void generateAllVersions(String versionlessCoordinates)
-      throws IOException, TemplateException, RepositoryException, URISyntaxException,
+      throws IOException,
+          TemplateException,
+          RepositoryException,
+          URISyntaxException,
           MavenRepositoryException {
     List<String> elements = Splitter.on(':').splitToList(versionlessCoordinates);
     checkArgument(

java-cloud-bom/libraries-release-data/pom.xml

@@ -22,6 +22,7 @@
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <maven.compiler.source>1.8</maven.compiler.source>
         <maven.compiler.target>1.8</maven.compiler.target>
+        <checkstyle.skip>true</checkstyle.skip>
     </properties>
 
     <build>

java-cloud-bom/pom.xml

@@ -27,9 +27,17 @@
     </license>
   </licenses>
   <modules>
-    <!-- Only modules to be published to Maven Central. No tests or release note generation. -->
+    <!-- Modules in the reactor. -->
     <module>google-cloud-bom</module>
     <module>libraries-bom</module>
+
+    <!-- maven.deploy.skip is set to true in the following modules so that they are not published to Maven Central -->
+    <module>dashboard</module>
+    <module>release-note-generation</module>
+    <module>libraries-release-data</module>
+    <module>tests</module>
+    <module>tests/dependency-convergence</module>
+    <module>tests/validate-bom</module>
   </modules>
 
   <build>

java-cloud-bom/release-note-generation/pom.xml

@@ -18,6 +18,7 @@
 		<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
 		<maven.compiler.source>11</maven.compiler.source>
 		<maven.compiler.target>11</maven.compiler.target>
+		<checkstyle.skip>true</checkstyle.skip>
 	</properties>
 
 	<dependencies>

java-cloud-bom/tests/dependency-convergence/pom.xml

@@ -13,6 +13,10 @@
   <organization>
     <name>Google</name>
   </organization>
+  <properties>
+    <maven.deploy.skip>true</maven.deploy.skip>
+    <checkstyle.skip>true</checkstyle.skip>
+  </properties>
   <licenses>
     <license>
       <name>Google Cloud Software License</name>
@@ -151,10 +155,7 @@
       <artifactId>google-cloud-dataplex</artifactId>
       <groupId>com.google.cloud</groupId>
     </dependency>
-    <dependency>
-      <artifactId>google-cloud-debugger-client</artifactId>
-      <groupId>com.google.cloud</groupId>
-    </dependency>
+
     <dependency>
       <artifactId>google-cloud-deploy</artifactId>
       <groupId>com.google.cloud</groupId>
@@ -326,10 +327,7 @@
       <artifactId>google-cloud-redis</artifactId>
       <groupId>com.google.cloud</groupId>
     </dependency>
-    <dependency>
-      <artifactId>google-cloud-resource-settings</artifactId>
-      <groupId>com.google.cloud</groupId>
-    </dependency>
+
     <dependency>
       <artifactId>google-cloud-resourcemanager</artifactId>
       <groupId>com.google.cloud</groupId>

java-cloud-bom/tests/pom.xml

@@ -22,6 +22,8 @@
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
     <maven.compiler.target>1.8</maven.compiler.target>
     <maven.compiler.source>1.8</maven.compiler.source>
+    <maven.deploy.skip>true</maven.deploy.skip>
+    <checkstyle.skip>true</checkstyle.skip>
   </properties>
   <licenses>
     <license>

java-cloud-bom/tests/validate-bom/pom.xml

@@ -19,6 +19,8 @@
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
     <maven.compiler.source>11</maven.compiler.source>
     <maven.compiler.target>11</maven.compiler.target>
+    <maven.deploy.skip>true</maven.deploy.skip>
+    <checkstyle.skip>true</checkstyle.skip>
   </properties>
 
   <dependencies>