SonarQube’s approval of a PR is needlessly verbose and filled with nonsense.
A simple “Passes SonarQube inspections” would be lovely.
Colin
December 10, 2019, 7:08am
2
As much as that sounds like something we would write, not actually our copy! That message comes from Mibex integrations between SonarQube and Bitbucket, so take it up with them (or checkout our first party Bitbucket Server PR Decoration )
Ah yes!
else
FailingBuildStatus(Severity.valueOf(pluginConfiguration.sonarApprovalSeverityLevel()), issuesWithAboveMaxSeverity)
}
def formatAsMarkdown(): String = {
val markdown = new StringBuilder()
markdown.append(s"${SonarUtils.sonarMarkdownPrefix()} reported ")
newIssuesBySeverity.values.toList match {
case Nil =>
markdown.append("no issues. Take a chocolate :-)\n\n")
case xs =>
markdown.append(s"${xs.sum} ${StringUtils.pluralise("issue", xs.sum)}:\n\n")
Severity.values().reverse foreach { s =>
printNewIssuesForMarkdown(markdown, s)
}
markdown.append("\n\nWatch the comments in this pull request to review them. ")
}
appendIssueSeverityRemark(markdown)
markdown.toString()
Thanks!
