diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index cddb7e3..50735d9 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -1,20 +1,30 @@ -# Maintainers for terminal-arcade. +# Who reviews what. # -# Anyone listed here is automatically requested for review when a matching file -# changes. This is what gives the 48-hour review promise an actual owner rather -# than "someone will get to it". +# GitHub requests a review from these people automatically when a pull request +# touches a path they own. Code-owner review is REQUIRED on this repository, so +# one of the names on the matching line has to approve before anything merges. # -# A handle only works if that person has write access to this repo AND has -# accepted the invitation. Check Settings → Collaborators. +# Maintainer for the Python terminal games: @Vedant-OGC +# +# @pushpam2404 is on every line on purpose: +# - GitHub does not let anybody approve their own pull request, so a line +# naming one person would deadlock the moment that person opened one +# - if the maintainer is unavailable, nothing has to wait for them +# +# The last matching line wins, so the more specific rules are at the bottom. # Everything, by default -* @pushpam2404 @RustyLegend @vishal-vetcha @Prashansure +* @Vedant-OGC @pushpam2404 -# CI and contribution policy — these decide what "passing" means, so they get -# a narrower set of reviewers. +# One file per game — where most contributions land +/arcade/ @Vedant-OGC @pushpam2404 +/tests/ @Vedant-OGC @pushpam2404 + +# Automation and the contribution rules. These decide what the checks are, so a +# pull request must not be able to change them with only its own author's +# approval. /.github/ @pushpam2404 /CONTRIBUTING.md @pushpam2404 /CODE_OF_CONDUCT.md @pushpam2404 - -# The games themselves are the safest place for a newer maintainer to review -/arcade/games/ @pushpam2404 @RustyLegend @vishal-vetcha @Prashansure +/SECURITY.md @pushpam2404 +/LICENSE @pushpam2404