Skip to content
Permalink
Browse files

690: Merge commit message is not Merge jdk, but: Merge jdk

Reviewed-by: rwestberg
  • Loading branch information
Erik Helin
Erik Helin committed Sep 18, 2020
1 parent ae829d6 commit 87ac6537a17f5b7368492a9658bd0db26100824b
@@ -128,7 +128,16 @@ public void visit(InvalidReviewersIssue e) {
@Override
public void visit(MergeMessageIssue e) {
var message = String.join("\n", e.commit().message());
addFailureMessage(e.check(), "Merge commit message is not " + e.expected() + ", but: " + message);
var desc = "Merge commit message is not `" + e.expected() + "`, but:";
if (e.commit().message().size() == 1) {
desc += " `" + message + "`";
} else {
desc += "\n" +
"```\n" +
message +
"```";
}
addFailureMessage(e.check(), desc);
}

@Override

1 comment on commit 87ac653

@bridgekeeper

This comment has been minimized.

Copy link

@bridgekeeper bridgekeeper bot commented on 87ac653 Sep 18, 2020

Please sign in to comment.
You can’t perform that action at this time.