Addresses regression with jenkins setup that does not use the multiproject setup

The Jenkins Gitlab Hook Plugin allows for two kinds of setups. One where a Jenkins job is created for each branch and one where all branches are built in a single Jenkins job. MR !9 (merged) added support for the multi-job setup but broke the single job setup. This request addresses #30 (closed) and fixes the regression.

mentioned in issue #30 (closed)

LGTM; thanks @chrisrohr.

Milestone changed to 8.0

Reassigned to @dzaporozhets

Thank you @chrisrohr for submitting a fix.

@dzaporozhets can you give us a hand and review the code?

@rspeicher I think this is patch release material, do you agree?

@chrisrohr The rest looks good! Can you make a fix for my comment and confirm that tests are green so we can merge it?

Added 1 commit:

• f80ca51b - Cleaned up conditional syntax

@dzaporozhets I cleaned up the conditional and manually have confirmed the tests, regardless I setup CI and the build is running.

@chrisrohr awesome! Thank you

Status changed to merged

mentioned in commit 086bf47e

@chrisrohr btw you might need to click Enable shared runners button to make builds running

Thank you for reviewing and merging @dzaporozhets

@rspeicher can you make sure this gets added to the next patch release?

@patricio Will do, 8.0.4.

@patricio and @dzaporozhets Thanks for bringing in my fix. One note just so you are aware. There seemed to have slipped into this another patch I was going to submit once this got in. If you look at the changes of my last commit, there is another option added to the screen to allow the unstable jenkins builds to be configured to show as failed or success. This is related to !11 (merged) and !18 (closed) that were merged in a while back. I apologize for this oversight, but the extra code did include tests that passed.

@chrisrohr yes thank you for mention it. Code looks good and have test so should be safe

Picked into 8-0-stable-ee.

mentioned in commit 9dd93813

Mentioned in commit 086bf47e