fix: exclude google-cloud-firestore-admin (#1157)

stephaniewang526 · web-flow · commit 50f3dfb346e6 · 2020-11-09T12:49:02.000-05:00
fixes #1150
diff --git a/dashboard/src/main/java/com/google/cloud/tools/opensource/cloudbomdashboard/DashboardMain.java b/dashboard/src/main/java/com/google/cloud/tools/opensource/cloudbomdashboard/DashboardMain.java
@@ -149,9 +149,11 @@ private static Path generate(Bom bom) throws IOException, TemplateException, URI
       // All dependencies with google-cloud-shared-dependencies must have group ID
       // "com.google.cloud". google-cloud-core and bigtable-emulator are unique
       // in that they do not require google-cloud-shared-dependencies.
+      // google-cloud-firestore-admin is a module of google-cloud-firestore
       if ("com.google.cloud".equals(artifact.getGroupId())
           && !artifact.getArtifactId().contains("google-cloud-core")
-          && !artifact.getArtifactId().contains("google-cloud-bigtable-emulator")) {
+          && !artifact.getArtifactId().contains("google-cloud-bigtable-emulator")
+          && !artifact.getArtifactId().contains("google-cloud-firestore-admin")) {
         managedDependencies.add(artifact);
       }
     }

Original file line number	Diff line number	Diff line change
`@@ -149,9 +149,11 @@ private static Path generate(Bom bom) throws IOException, TemplateException, URI`
`149`	`149`	`// All dependencies with google-cloud-shared-dependencies must have group ID`
`150`	`150`	`// "com.google.cloud". google-cloud-core and bigtable-emulator are unique`
`151`	`151`	`// in that they do not require google-cloud-shared-dependencies.`
	`152`	`+ // google-cloud-firestore-admin is a module of google-cloud-firestore`
`152`	`153`	`if ("com.google.cloud".equals(artifact.getGroupId())`
`153`	`154`	`&& !artifact.getArtifactId().contains("google-cloud-core")`
`154`		`- && !artifact.getArtifactId().contains("google-cloud-bigtable-emulator")) {`
	`155`	`+ && !artifact.getArtifactId().contains("google-cloud-bigtable-emulator")`
	`156`	`+ && !artifact.getArtifactId().contains("google-cloud-firestore-admin")) {`
`155`	`157`	`managedDependencies.add(artifact);`
`156`	`158`	`}`
`157`	`159`	`}`