New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding suport to stop build api #29
Conversation
callback(error, response); | ||
return; | ||
} | ||
var data = "job is stoped"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
stopped*
Would be great to add this command to the README! |
@jansepar |
@@ -78,6 +79,19 @@ var init = exports.init = function(host, options) { | |||
callback(null, data); | |||
}); | |||
}, | |||
stopBuild: function(jobname, buildNumber, callback) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could this match the format of other commands and variable using the _
to denote spaces? So stopBuild
→ stop_build
? Obviously up to @jansepar, but it looks weird to mix formats to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@brianmichel bd015e0 fix this. thank you.
Hey @jansepar what's going on with this PR, can it be merged? |
Hi guys, sorry for the delay on this, it's looking good thanks! 👍 |
Adding suport to stop build api
No description provided.