diff options
| author | Douwe Maan <douwe@gitlab.com> | 2018-01-29 22:06:34 +0300 |
|---|---|---|
| committer | Douwe Maan <douwe@gitlab.com> | 2018-01-29 22:06:34 +0300 |
| commit | 7a18675c59d46da7234dd0cdf6960b527b513a80 (patch) | |
| tree | c2847376bbe2195ea43e40992302030fea2dcf76 /lib | |
| parent | 86dc598518b1e4dd5a5c02ac8e0cb6930e7e36c4 (diff) | |
| parent | 395e500f0143258c956f11e09dda676d7ed2b554 (diff) | |
Merge branch 'osw-consistent-values-on-api-merge-status' into 'master'
Return more consistent values for merge_status on MR API
Closes #20639
See merge request gitlab-org/gitlab-ce!16708
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/api/entities.rb | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/lib/api/entities.rb b/lib/api/entities.rb index 7b9a80a234b..cb222697f32 100644 --- a/lib/api/entities.rb +++ b/lib/api/entities.rb @@ -507,7 +507,15 @@ module API expose :work_in_progress?, as: :work_in_progress expose :milestone, using: Entities::Milestone expose :merge_when_pipeline_succeeds - expose :merge_status + + # Ideally we should deprecate `MergeRequest#merge_status` exposure and + # use `MergeRequest#mergeable?` instead (boolean). + # See https://gitlab.com/gitlab-org/gitlab-ce/issues/42344 for more + # information. + expose :merge_status do |merge_request| + merge_request.check_if_can_be_merged + merge_request.merge_status + end expose :diff_head_sha, as: :sha expose :merge_commit_sha expose :user_notes_count |