Commit 7e813afd authored by Mayra Cabrera's avatar Mayra Cabrera

Merge branch 'kerrizor/remove-codeowners_match_ancestor_groups-flag' into 'master'

Remove codeowners_match_ancestor_groups feature flag

See merge request gitlab-org/gitlab!43606
parents 379ab4e9 b5ea36f0
---
name: codeowners_match_ancestor_groups
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/31887
rollout_issue_url:
group: group::source code
type: development
default_enabled: true
......@@ -24,22 +24,18 @@ module Gitlab
groups = project.invited_groups.where_full_path_in(extractor.names)
if Feature.enabled?(:codeowners_match_ancestor_groups, default_enabled: true)
group_list = groups.with_route.with_users.to_a
if project.group
# If the project.group's ancestor group(s) are listed as owners, add
# them to group_list
#
if applicable_ancestors(extractor.names).any?
group_list.concat(applicable_ancestors(extractor.names))
end
group_list = groups.with_route.with_users.to_a
if project.group
# If the project.group's ancestor group(s) are listed as owners, add
# them to group_list
#
if applicable_ancestors(extractor.names).any?
group_list.concat(applicable_ancestors(extractor.names))
end
group_list.uniq
else
groups.with_route.with_users
end
group_list.uniq
end
def applicable_ancestors(extractor_names)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment