From 707ecb26da5958739b863a674e8d411575b6ac79 Mon Sep 17 00:00:00 2001 From: Jeff Hodges Date: Mon, 15 Jul 2024 21:15:24 -0700 Subject: [PATCH] create working CODEOWNERS file We would like to have code reviewers automatically alerted and assigned when PRs are posted to autograph codebases. This is the next step towards that for the autograph repo. The next step will be to enable the automatic assignation in the GitHub repo settings (one day, we'll have terraform for those settings, I hope). Autograph is going to use the @mozilla-services/autograph-devs GitHub team to provide automatic assignation of code reviewers in autograph. These folks are expected to provide timely reviews of code coming into autograph. As a back up and in order to provide some escape valves, we also have a `@mozilla-services/autograph-mergers`. This team is folks who are trusted to merge code but can't provide timely reviews like our SREs. This may not be a team that we keep around in the future, but given our current team size and volatility, it's a good back up. Finally, the `addons.mozilla.org` teams has previously requested alerts when code is changed in `signer/xpi`. We provide that here using the same GitHub team as the one used in the mozilla/addons-server CODEOWNERS. Updates AUT-18 --- .github/CODEOWNERS | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 2b5fde616..57c059744 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -1,8 +1,3 @@ -* @g-k, @kkleemola, @pwnbus +* @mozilla-services/autograph-devs @mozilla-services/autograph-mergers -# the AMO team should review changes that impact Addon signing -signer/xpi/ @willdurand - -# TODO: check with Mathieu -# signer/contentsignature @leplatrem -# signer/contentsignaturepki @leplatrem +signer/xpi/ @mozilla-services/autograph-devs @mozilla-services/autograph-mergers @mozilla/addons-engineering \ No newline at end of file