Skip to content

Use name (path) as the category if codeowners lacks section

Lin Jen-Shin requested to merge use-name-for-unnamed-codeowners into master

What does this MR do and why?

This partially solves !166 (comment 1160560133) when we have CODEOWNERS rules that don't have a section.

This will change the table in gitlab-org/gitlab!102961 (comment 1159932311)

From:

Category Reviewer Maintainer
~"codeowners" Reviewer review is optional for ~"codeowners" Diana Logan current availability (@dianalogan) (UTC+0, 9 hours behind @marcel.amirault)

To:

Category Reviewer Maintainer
/data/deprecations/templates/ Reviewer review is optional for /data/deprecations/templates/ Diana Logan current availability (@dianalogan) (UTC+0, 9 hours behind @marcel.amirault)
/data/deprecations/*.yml Reviewer review is optional for /data/deprecations/*.yml No maintainer available

It's not ideal, but I think it's better. To further improve this, we'll have to update https://gitlab.com/gitlab-org/gitlab/blob/master/.gitlab/CODEOWNERS

MR acceptance checklist

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

Edited by Lin Jen-Shin

Merge request reports