Merge pull request #468 from gsmet/build-reporter-error

Include Build Reporter line when it's failing
quarkusio · Aug 1, 2024 · ec91720 · ec91720
2 parents 537e9e2 + 935002c
commit ec91720
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/main/java/io/quarkus/bot/workflow/report/QuarkusWorkflowReportJobIncludeStrategy.java b/src/main/java/io/quarkus/bot/workflow/report/QuarkusWorkflowReportJobIncludeStrategy.java
@@ -13,15 +13,15 @@ public class QuarkusWorkflowReportJobIncludeStrategy implements WorkflowReportJo
 
     @Override
     public boolean include(WorkflowReport report, WorkflowReportJob job) {
-        if (QuarkusWorkflowConstants.JOB_NAME_BUILD_REPORT.equals(job.getName())) {
-            return false;
-        }
         if (job.getName().startsWith(WorkflowConstants.BUILD_SUMMARY_CHECK_RUN_PREFIX)) {
             return false;
         }
         if (job.isFailing()) {
             return true;
         }
+        if (QuarkusWorkflowConstants.JOB_NAME_BUILD_REPORT.equals(job.getName())) {
+            return false;
+        }
 
         // in this particular case, we exclude the Windows job as it does not run the containers job
         // (no Docker support on Windows) and thus does not provide a similar coverage as the Linux