Add "Required" title to jcheck-check only if it fails

Reviewed-by: rwestberg
openjdk · Nov 6, 2019 · 43f8ff2 · 43f8ff2
1 parent 0d73a1f
commit 43f8ff2
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/bots/pr/src/main/java/org/openjdk/skara/bots/pr/CheckRun.java b/bots/pr/src/main/java/org/openjdk/skara/bots/pr/CheckRun.java
@@ -171,6 +171,7 @@ private void updateCheckBuilder(CheckBuilder checkBuilder, PullRequestCheckIssue
         if (visitor.isReadyForReview() && additionalErrors.isEmpty()) {
             checkBuilder.complete(true);
         } else {
+            checkBuilder.title("Required");
             var summary = Stream.concat(visitor.getMessages().stream(), additionalErrors.stream())
                                 .sorted()
                                 .map(m -> "- " + m)
@@ -448,7 +449,6 @@ private void updateMergeReadyComment(boolean isReady, String commitMessage, List
 
     private void checkStatus() {
         var checkBuilder = CheckBuilder.create("jcheck", pr.headHash());
-        checkBuilder.title("Required");
         var censusDomain = censusInstance.configuration().census().domain();
         Exception checkException = null;