Assign approvers based on code owners
Compare changes
Files
4- Mark Chao authored
+ 2
− 2
@@ -35,8 +35,8 @@ class MergeRequest < ActiveRecord::Base
The CE port is at https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/22513
Assign approvers based on code owners
The major changes are:
overall_approvers
method now returns User
instead of Approver
. This is because it needs to include code owners, which are dynamically calculated therefore does not have a corresponding Approver
column.overall_approvers
are changed to work with User
instead.user_id
instead. This is because we only have User
instead of Approver
, and if we want to get an user's approver it would cause extra database queries. At the end I decided to allow deleting of approvers by using user_id
instead.Closes #1012 (closed)
/ee
folder