diff options
| author | Stan Hu <stanhu@gmail.com> | 2019-03-01 23:29:04 -0800 |
|---|---|---|
| committer | Stan Hu <stanhu@gmail.com> | 2019-03-02 06:31:09 -0800 |
| commit | b6a972787048a2ab9024051ecfe40f1a46bcc67d (patch) | |
| tree | 94f1db9a8bcd3533ff8e5e0f05a27f4f702b40de /app/services/merge_requests | |
| parent | 5c9aa45489aee7cd1edaa15a30fb8a3aa24417a8 (diff) | |
| download | gitlab-ce-b6a972787048a2ab9024051ecfe40f1a46bcc67d.tar.gz | |
Show pre-receive error message in merge request widget
When a merge fails due to a locked file, no feedback was given to the
user as to why it failed if the user used the Merge button. However, if
the push attempt happened directly in the Web UI, the pre-receive
message would be displayed.
To be consistent and to make it clear why a merge failed, we now include
the pre-receive error message in the merge request widget.
Closes https://gitlab.com/gitlab-org/gitlab-ee/issues/10165
Diffstat (limited to 'app/services/merge_requests')
| -rw-r--r-- | app/services/merge_requests/merge_service.rb | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/app/services/merge_requests/merge_service.rb b/app/services/merge_requests/merge_service.rb index 8241e408ce5..b29e0b1759e 100644 --- a/app/services/merge_requests/merge_service.rb +++ b/app/services/merge_requests/merge_service.rb @@ -76,8 +76,7 @@ module MergeRequests def try_merge repository.merge(current_user, source, merge_request, commit_message) rescue Gitlab::Git::PreReceiveError => e - handle_merge_error(log_message: e.message) - raise_error('Something went wrong during merge pre-receive hook') + raise_error("Something went wrong during merge pre-receive hook: #{e.message}") rescue => e handle_merge_error(log_message: e.message) raise_error('Something went wrong during merge') |