Skip to content

Commit d50988f

Browse files
pmacalexgibson
authored andcommitted
Fix slack failure notifications
They were using the sparkels emoji because it was expecting a status of "failure" instead of "failed" and sparkels is the default.
1 parent 5b277c9 commit d50988f

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

Jenkinsfile

+3-3
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ def buildSite(destination) {
55
try {
66
sh "bin/build.sh " + destination
77
} catch(err) {
8-
sh "bin/slack-notify.sh --stage 'build " + env.BRANCH_NAME + "' --status 'failed'"
8+
sh "bin/slack-notify.sh --stage 'build " + env.BRANCH_NAME + "' --status failure"
99
throw err
1010
}
1111
}
@@ -16,10 +16,10 @@ def syncS3(String bucket) {
1616
try {
1717
sh "cd dist && aws s3 sync . s3://" + bucket + " --acl public-read --delete --profile protocol"
1818
} catch(err) {
19-
sh "bin/slack-notify.sh --stage 's3 sync " + env.BRANCH_NAME + "' --status 'failed'"
19+
sh "bin/slack-notify.sh --stage 's3 sync " + env.BRANCH_NAME + "' --status failure"
2020
throw err
2121
}
22-
sh "bin/slack-notify.sh --stage 's3 sync " + env.BRANCH_NAME + "' --status 'shipped'"
22+
sh "bin/slack-notify.sh --stage 's3 sync " + env.BRANCH_NAME + "' --status shipped"
2323
}
2424
}
2525

0 commit comments

Comments
 (0)