Commit b5b1eab1 authored by Stan Hu's avatar Stan Hu

Merge branch 'id-mark-optional_code_owners_sections-as-default' into 'master'

Enable optional_code_owners_sections by default

See merge request gitlab-org/gitlab!51643
parents a19ada1c f17a67eb
---
title: Designate optional sections in the codeowners file
merge_request: 51643
author:
type: added
...@@ -5,4 +5,4 @@ rollout_issue_url: ...@@ -5,4 +5,4 @@ rollout_issue_url:
milestone: '13.8' milestone: '13.8'
type: development type: development
group: group::source code group: group::source code
default_enabled: false default_enabled: true
...@@ -22,7 +22,7 @@ class ApprovalWrappedCodeOwnerRule < ApprovalWrappedRule ...@@ -22,7 +22,7 @@ class ApprovalWrappedCodeOwnerRule < ApprovalWrappedRule
end end
def section_optional? def section_optional?
return false unless Feature.enabled?(:optional_code_owners_sections, project) return false unless Feature.enabled?(:optional_code_owners_sections, project, default_enabled: true)
Gitlab::CodeOwners.optional_section?(project, merge_request.target_branch, section) Gitlab::CodeOwners.optional_section?(project, merge_request.target_branch, section)
end end
......
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