Skip to content

Commit 4c680dc

Browse files
yrodieremarko-bekhta
authored andcommitted
Direct link to failure in Develocity build scan reports
1 parent a4833bd commit 4c680dc

File tree

2 files changed

+10
-5
lines changed

2 files changed

+10
-5
lines changed

src/main/java/org/hibernate/infra/bot/ExtractDevelocityBuildScans.java

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -167,6 +167,7 @@ else if ( customValue.getName().equals( "CI stage" ) ) {
167167
? DevelocityCIBuildScan.Status.FAILURE
168168
: DevelocityCIBuildScan.Status.SUCCESS,
169169
buildScanURI,
170+
deploymentConfig.develocity().uri().resolve( "/s/" + build.getId() + "/failure" ),
170171
deploymentConfig.develocity().uri().resolve( "/s/" + build.getId() + "/tests" ),
171172
deploymentConfig.develocity().uri().resolve( "/s/" + build.getId() + "/console-log" )
172173
);
@@ -235,16 +236,20 @@ private String formatBuildScanList(List<DevelocityCIBuildScan> buildScans) {
235236
summary.append( "| :-: | -- | -- | -- | :-: | :-: | :-: |\n" );
236237
for ( DevelocityCIBuildScan buildScan : buildScans ) {
237238
summary.append(
238-
"| %s | `%s` | `%s` | `%s` | [:mag:](%s) | [%s](%s) | [:page_with_curl:](%s) |\n"
239+
"| [%s](%s) | `%s` | `%s` | `%s` | [:mag:](%s) | [%s](%s) | [:page_with_curl:](%s) |\n"
239240
.formatted(
240241
statusToEmoji( buildScan.status() ),
242+
switch ( buildScan.status() ) {
243+
case SUCCESS -> buildScan.buildScanUri();
244+
case FAILURE -> buildScan.failuresUri();
245+
},
241246
String.join( " ", buildScan.jobOrWorkflow(), buildScan.stage() ),
242247
String.join( "` `", buildScan.tags() ),
243248
String.join( " ", buildScan.goals() ),
244-
buildScan.buildScan(),
249+
buildScan.buildScanUri(),
245250
statusToEmoji( buildScan.testStatus() ),
246-
buildScan.tests(),
247-
buildScan.logs()
251+
buildScan.testsUri(),
252+
buildScan.logsUri()
248253
) );
249254
}
250255
return summary.toString();

src/main/java/org/hibernate/infra/bot/develocity/DevelocityCIBuildScan.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
public record DevelocityCIBuildScan(String provider, String jobOrWorkflow, String stage,
88
Long availableAt, List<String> tags, List<String> goals,
99
Status status, Status testStatus,
10-
URI buildScan, URI tests, URI logs) {
10+
URI buildScanUri, URI failuresUri, URI testsUri, URI logsUri) {
1111

1212
public static final Comparator<? super DevelocityCIBuildScan> COMPARATOR =
1313
Comparator.comparing( DevelocityCIBuildScan::provider )

0 commit comments

Comments
 (0)