Skip to content

Display code owners info (move code to EE)

Jacques Erasmus requested to merge 335378-move-code-owners-to-ee into master

What does this MR do and why?

This is a follow-up from !74298 (comment 744394455) which got reverted because of a spec failure.

The EE-only constant Gitlab::CodeOwners was being referred to by a core class, this moves the code from core to EE.

Screenshots or screen recordings

before after
Screenshot_2021-11-11_at_14.05.16 Screenshot_2021-11-11_at_13.45.55
Screenshot_2021-11-11_at_13.47.31
Screenshot_2021-11-11_at_14.03.32

How to set up and validate locally

  1. Enable the refactor FF

    Feature.enable(:refactor_blob_viewer)

  2. Create a CODEOWNERS file

  3. Populate the CODEOWNERS file

  4. Open a file that has codeowners

  5. Notice the codeowners section

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #335378 (closed)

Edited by Jacques Erasmus

Merge request reports