Skip to content

Commit 8c986a6

Browse files
committed
Report Quarkus platform version as N/A when there are no platform BOMs detected and CUSTOM when platform BOMs are detected but quarkus-bom is not among them
1 parent d2f3938 commit 8c986a6

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

independent-projects/tools/analytics-common/src/main/java/io/quarkus/analytics/AnalyticsService.java

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@
4242
import java.time.Instant;
4343
import java.time.ZoneId;
4444
import java.time.format.TextStyle;
45+
import java.util.Collection;
4546
import java.util.List;
4647
import java.util.Locale;
4748
import java.util.Map;
@@ -258,10 +259,15 @@ private boolean allEnvSet(String... names) {
258259
}
259260

260261
private String getQuarkusVersion(ApplicationModel applicationModel) {
261-
return applicationModel.getPlatforms().getImportedPlatformBoms().stream()
262+
final Collection<ArtifactCoords> platformBoms = applicationModel.getPlatforms().getImportedPlatformBoms();
263+
if (platformBoms.isEmpty()) {
264+
// Typically, this situation should result in a build error, but it's not up to this service to fail it
265+
return "N/A";
266+
}
267+
return platformBoms.stream()
262268
.filter(artifactCoords -> artifactCoords.getArtifactId().equals("quarkus-bom"))
263269
.map(ArtifactCoords::getVersion)
264270
.findFirst()
265-
.orElse("N/A");
271+
.orElse("CUSTOM");
266272
}
267273
}

0 commit comments

Comments
 (0)