This is an automated email from the ASF dual-hosted git repository.

hboutemy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-dist-tool.git

commit 47b5cf8915f0cfa66e87b192a7f1247e93c5cee3
Author: HervĂ© Boutemy <[email protected]>
AuthorDate: Tue Nov 11 10:42:06 2025 +0100

    extract render method
---
 .../tools/jobs/master/ListMasterJobsReport.java    | 40 +++++++++++-----------
 1 file changed, 20 insertions(+), 20 deletions(-)

diff --git 
a/src/main/java/org/apache/maven/dist/tools/jobs/master/ListMasterJobsReport.java
 
b/src/main/java/org/apache/maven/dist/tools/jobs/master/ListMasterJobsReport.java
index a2a3845..d9ef16c 100644
--- 
a/src/main/java/org/apache/maven/dist/tools/jobs/master/ListMasterJobsReport.java
+++ 
b/src/main/java/org/apache/maven/dist/tools/jobs/master/ListMasterJobsReport.java
@@ -141,26 +141,7 @@ public class ListMasterJobsReport extends 
AbstractJobsReport {
                     int size = e.getValue().size();
                     sink.text(size + " job" + (size > 1 ? "s" : "") + " with 
status " + e.getKey() + ":");
                     sink.list();
-                    e.getValue().forEach(r -> {
-                        sink.listItem();
-                        sink.rawText(r.getIcon());
-
-                        sink.rawText("<span");
-                        if ((r.getLastBuild() == null)
-                                || 
r.getLastBuild().isBefore(ZonedDateTime.now().minusMonths(1))) {
-                            sink.rawText(" style=\"color:red\"");
-                        }
-                        sink.rawText(">("
-                                + ((r.getLastBuild() == null)
-                                        ? "-"
-                                        : 
r.getLastBuild().format(DateTimeFormatter.ISO_LOCAL_DATE))
-                                + ")</span> ");
-
-                        sink.link(r.getBuildUrl());
-                        sink.rawText(r.getRepositoryName());
-                        sink.link_();
-                        sink.listItem_();
-                    });
+                    e.getValue().forEach(r -> renderJobResult(sink, r));
                     sink.list_();
 
                     sink.listItem_();
@@ -170,6 +151,25 @@ public class ListMasterJobsReport extends 
AbstractJobsReport {
         sink.body_();
     }
 
+    private void renderJobResult(Sink sink, Result r) {
+        sink.listItem();
+        sink.rawText(r.getIcon());
+
+        sink.rawText("<span");
+        if ((r.getLastBuild() == null)
+                || 
r.getLastBuild().isBefore(ZonedDateTime.now().minusMonths(1))) {
+            sink.rawText(" style=\"color:red\"");
+        }
+        sink.rawText(">("
+                + ((r.getLastBuild() == null) ? "-" : 
r.getLastBuild().format(DateTimeFormatter.ISO_LOCAL_DATE))
+                + ")</span> ");
+
+        sink.link(r.getBuildUrl());
+        sink.rawText(r.getRepositoryName());
+        sink.link_();
+        sink.listItem_();
+    }
+
     private Comparator<String> resultComparator() {
         final List<String> orderedStatus = Arrays.asList("FAILURE", 
"UNSTABLE", "UNKNOWN", "SUCCESS");
         return (l, r) -> {

Reply via email to