Skip to content

Commit

Permalink
create working CODEOWNERS file
Browse files Browse the repository at this point in the history
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. This team will be configured to be selected from for reviews.

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
  • Loading branch information
jmhodges committed Jul 16, 2024
1 parent dd722f2 commit 7b2c36d
Showing 1 changed file with 2 additions and 7 deletions.
9 changes: 2 additions & 7 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -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

0 comments on commit 7b2c36d

Please sign in to comment.