Skip to content

Feature/include all prs#26

Closed
osbornk wants to merge 3 commits intojenkinsci:masterfrom
centraldesktop:feature/include_all_prs
Closed

Feature/include all prs#26
osbornk wants to merge 3 commits intojenkinsci:masterfrom
centraldesktop:feature/include_all_prs

Conversation

@osbornk
Copy link

@osbornk osbornk commented Nov 14, 2016

We need this to include feature branches as well as all pull requests. So, we have the following pattern.

Branch includes: /feature//master /release/

But we also want to build all pull requests. So a new checkbox for "Include all pull requests" was added.

We did this pattern because we were finding that when we pushed a commit to a branch that had an associated pull request, Jenkins was building twice. Once for the pull request and another for the branch associated with that pull request. That was putting a lot of strain on our Jenkins.

I also fixed the pattern notation to allow a wildcard in any position, not just the beginning or end.

@tknerr
Copy link

tknerr commented Nov 22, 2016

see also #9

@tknerr tknerr mentioned this pull request Jan 10, 2017
6 tasks
@stephenc
Copy link
Member

All UI changes are being blocked until I complete the parity UI config with the new UI config I am implementing to be shared by Bitbucket and GitHub. Here is the GitHub preview

jenkins-43507

@bhipple
Copy link

bhipple commented May 24, 2017

Any progress on this one? I'm running into the include branches bug and would like to have this if possible.

@stephenc
Copy link
Member

Will be considered after the combined GitHub/Bitbucket UI refactoring to use common shared behaviours (and may even be fixed by that refactoring)

@stephenc
Copy link
Member

likely superseded by #53

@stephenc stephenc closed this Jun 12, 2017
fengxx pushed a commit to fengxx/bitbucket-branch-source-plugin that referenced this pull request Mar 13, 2018
[JENKINS-32493] Adapt to Parent POM 2.3.
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.

4 participants