Skip to content

Update merge_requests.rb

username-removed-749943 requested to merge pwmckenna/gitlab-ce:patch-7 into master

I can't get gitlab running locally to test this, but we've made this change to our internal gitlab instance and it works. If this is too rough looking just close.

What does this MR do?

fix for https://gitlab.com/gitlab-org/gitlab-ce/issues/22740

Merges prs that have successful builds, if during the pipeline a request to merge was made with the merge_when_build_succeeds flag is set.

Are there points in the code the reviewer needs to double check?

Why was this MR needed?

Merge requests were failing due to running pipelines, even with merge_when_build_succeeds was set, and the pipelines went on to succeed.

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

https://gitlab.com/gitlab-org/gitlab-ce/issues/22740

Merge request reports