Skip to content

Commit 3b2867a

Browse files
authored
Merge pull request #15 from envoy/add-failure-reason-to-job-ignore
Update job_ignore to accept a reason
2 parents 1259d4f + 7449b55 commit 3b2867a

File tree

1 file changed

+2
-8
lines changed

1 file changed

+2
-8
lines changed

lib/response.js

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -54,16 +54,10 @@ Response = (function () {
5454
this._attachments.push(...attachments)
5555
this.meta('set_job_attachments', JSON.stringify(this._attachments))
5656
}
57-
Response.prototype.job_ignore = function (status, data) {
58-
this.meta('set_job_status', 'ignored')
59-
this.meta('set_job_status_message', status)
60-
if (data) {
61-
this._respond(data)
62-
}
63-
}
64-
Response.prototype.job_ignore = function (status, data) {
57+
Response.prototype.job_ignore = function (status, reason, data) {
6558
this.meta('set_job_status', 'ignored')
6659
this.meta('set_job_status_message', status)
60+
this.meta('set_job_failure_message', reason)
6761
if (data) {
6862
this._respond(data)
6963
}

0 commit comments

Comments
 (0)