Skip to content

Commit

Permalink
jobs/build: don't include build result as string in Slack msg
Browse files Browse the repository at this point in the history
It's redundant with the emojis and colour and isn't consistent with the
other job notifications.
  • Loading branch information
jlebon authored and dustymabe committed Dec 13, 2022
1 parent 3027dd6 commit a37d3f7
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions jobs/build-arch.Jenkinsfile
Original file line number Diff line number Diff line change
Expand Up @@ -417,11 +417,11 @@ lock(resource: "build-${params.STREAM}-${basearch}") {
// SUCCESS, but no new builds? Must've been a no-op
return
}
message = ":sparkles: ${message} - SUCCESS"
message = ":sparkles: ${message}"
} else if (currentBuild.result == 'UNSTABLE') {
message = ":warning: ${message} - WARNING"
message = ":warning: ${message}"
} else {
message = ":fire: ${message} - FAILURE"
message = ":fire: ${message}"
}

if (newBuildID) {
Expand Down
6 changes: 3 additions & 3 deletions jobs/build.Jenkinsfile
Original file line number Diff line number Diff line change
Expand Up @@ -509,11 +509,11 @@ lock(resource: "build-${params.STREAM}") {
// SUCCESS, but no new builds? Must've been a no-op
return
}
message = ":sparkles: ${message} - SUCCESS"
message = ":sparkles: ${message}"
} else if (currentBuild.result == 'UNSTABLE') {
message = ":warning: ${message} - WARNING"
message = ":warning: ${message}"
} else {
message = ":fire: ${message} - FAILURE"
message = ":fire: ${message}"
}

if (newBuildID) {
Expand Down

0 comments on commit a37d3f7

Please sign in to comment.