Adding CODEOWNERS

[hamzaremmal]

[skip ci]

[lrytz]

IIUC

• owners / owner teams will be notified and assigned as reviewers on PRs that change matching files
• we can set up a rule to require reviews from an owner to enable PR merging

What is the benefit over the status quo, it it solving a problem, or addressing some security concern?

[hamzaremmal]

What is the benefit over the status quo, it it solving a problem, or addressing some security concern?

So each line in the file will contain a rule, and the rule will consist of a glob and the team that is allowed to approve changes to matching files. For now, I'm suggesting to require the admins approval on sensible files and the infrastructure team for changes in the .github/workflows folder. Over the status quo, anybody can approve any changes so the proposed solution is more secure.

[mbovel]

I understand that some cleanup around the Scala organization was probably needed. However, my two cents is that we should be cautious about adding further restrictions just for the sake of it. The potential negative impact on usability and the time it takes to deal with these restrictions may outweigh the security benefits. From a community perspective, we should also make sure that these measures don’t lead to contributors feeling mistrusted or infantilized.