search

nemccarthy / stash-pullrequest-builder-plugin

A Jenkins plugin for Building Stash Pull Requests
https://wiki.jenkins-ci.org/display/JENKINS/Stash+pullrequest+builder+plugin
Other
64 stars 130 forks source link

Force the refs to be updated #32

Closed Karmaki closed 9 years ago

Karmaki commented 9 years ago

I experimented strange behaviors because in some cases, the plug-in was able to detect a new PR, but not to checkout the correct branch. I asked here : http://stackoverflow.com/questions/31458668/why-is-there-a-delay-between-a-pull-request-and-the-possibility-to-fetch-it-in, and someone from Stash explained the reason. It seems that you should call PullRequestService.canMerge before fetching. What do you thing ?

A temporary workaround is to force the update by exploring the PR in the UI, but it is not very convenient.

nemccarthy commented 9 years ago

I think we added this option already - check the Build only if PR is mergeable? & Build only if Stash reports no conflicts? options. This will wait until canMerge is true on the PR JSON response from stash if your instance takes a while to compute the PR branch.

Karmaki commented 9 years ago

OK. Thank you very much for your quick answer. I'll check these two options then. Many thanks for your great plug-in too !