Merge pull request #15 from envoy/add-failure-reason-to-job-ignore

Update job_ignore to accept a reason
envoy · Sep 20, 2018 · 3b2867a · 3b2867a
2 parents 1259d4f + 7449b55
commit 3b2867a
Showing 1 changed file with 2 additions and 8 deletions.
diff --git a/lib/response.js b/lib/response.js
@@ -54,16 +54,10 @@ Response = (function () {
     this._attachments.push(...attachments)
     this.meta('set_job_attachments', JSON.stringify(this._attachments))
   }
-  Response.prototype.job_ignore = function (status, data) {
-    this.meta('set_job_status', 'ignored')
-    this.meta('set_job_status_message', status)
-    if (data) {
-      this._respond(data)
-    }
-  }
-  Response.prototype.job_ignore = function (status, data) {
+  Response.prototype.job_ignore = function (status, reason, data) {
     this.meta('set_job_status', 'ignored')
     this.meta('set_job_status_message', status)
+    this.meta('set_job_failure_message', reason)
     if (data) {
       this._respond(data)
     }