-
Improvement
-
Resolution: Unresolved
-
Minor
-
None
Currently the text sent to Bitbucket as build status for a commit is hardcoded with messages that could be misleading (such as "This commit has test failures") if the job uses build statuses differently (i.e. UNSTABLE happens to mean something else). At the very least the text should say "The commit is unstable".
if (Result.SUCCESS.equals(result)) { status = new BitbucketBuildStatus(hash, "This commit looks good", "SUCCESSFUL", url, key, name); } else if (Result.UNSTABLE.equals(result)) { status = new BitbucketBuildStatus(hash, "This commit has test failures", "FAILED", url, key, name); } else if (Result.FAILURE.equals(result)) { status = new BitbucketBuildStatus(hash, "There was a failure building this commit", "FAILED", url, key, name); } else if (result != null) { // ABORTED etc. status = new BitbucketBuildStatus(hash, "Something is wrong with the build of this commit", "FAILED", url, key, name); } else { status = new BitbucketBuildStatus(hash, "The tests have started...", "INPROGRESS", url, key, name); }
- is related to
-
JENKINS-65741 Bitbucket commit status should include jenkins server URL
- In Review
- relates to
-
JENKINS-36755 Add the ability to switch off notifications.
- Resolved
-
JENKINS-35453 Allow disabling the default commit status notifier
- Resolved