Skip to content
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

Pull Request: Choose status which abort the flow build #67

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ueberhammDesign
Copy link

Reopen Pull-Request #59 to squash all together and resolve merge problems.
Original Message:

You can choose the status (Success, Unstable or Failure), which abort the flow build. It also contains, that the build can not be 'better' than any build inside the flow build.

@jenkinsadmin
Copy link
Member

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

@@ -91,6 +78,30 @@ abstract class DSLTestCase extends HudsonTestCase {
return flow.scheduleBuild2(0).get()
}

def runWithAbortWhenWorseThanSuccess = { script ->
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please refactor, I think you only need one method runWithAbortWhenWorseThan { script, abortWhenWorseThan -> ... }

@ueberhammDesign
Copy link
Author

New testcases added.

@dnozay
Copy link
Member

dnozay commented May 3, 2016

@ueberhammDesign

Since I am currently looking for someone else to take over as a maintainer; I will not be the one merging this change. Please check http://jenkins-ci.org/pull-request-greeting.

Side note: I think introducing new features in this plugin is risky and takes away resources that could be better spent on the pipeline plugin. However, please feel free to merge if you are interested in carrying on the flag.

I did look at your changes though, and thank you for addressing my earlier concerns.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants