Add CODEOWNERS

[Victor-Jung]

This PR adds a CODEOWNERS file where we can attribute code owners to specific parts of the project. All code owners of the files modified by a PR must review it. This provides an easy way to separate concerns for review and to control who can actually review the code.

Added

• A CODEOWNERS file

PR Merge Checklist

1. The PR is rebased on the latest devel commit and pointing to devel.
2. Your PR reviewed and approved.
3. All checks are passing.
4. The CHANGELOG.md file has been updated.

[Xeratec]

LGTM.
I am just wondering on what granularity you plan to assign code owners?

[FrancescoConti]

LGTM. I am just wondering on what granularity you plan to assign code owners?

IMHO owners should be clearly the two of you. If you want a Unibo person as well, my recommendation is @lukamac

[Victor-Jung]

@FrancescoConti I don't want to spam @lukamac each time a PR opens. @lukamac, if you really want to, I can add you ofc.

[lukamac]

I am a little dubious about the details. This looks to me like it is on the granularity of the repo. Would that mean who is in the CODEOWNERS file must approve each PR?

[Victor-Jung]

@Xeratec For now just you and I globally. Later, if some ppl really become experts of some distincts parts of the code (extension or platform for instance). Then we can start allocating at a finer granularity.

[Victor-Jung]

@lukamac Good question: with the proposed CODEOWNERS file Philip or I will need to approve the PR for it to be mergable.

[lukamac]

I would say, at this point in time, I am not as active in the PR review process so I would not put myself in there. If this changes over time, we can easily add me.
@FrancescoConti would you agree?

[FrancescoConti]

Sure, either way is fine for me