From 5cc7f46200c8f2b95d35fbecac636217f0e25aea Mon Sep 17 00:00:00 2001 From: teor Date: Thu, 3 Mar 2022 22:50:10 +1000 Subject: [PATCH] fix(codeowners): avoid multiple reviewer assignments (#3718) Assign more code to the largest review team, to avoid multiple reviewer assignments. --- .github/CODEOWNERS | 19 ++++++++++++------- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 2f2996a3..7b50047b 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -2,20 +2,25 @@ # this team will be requested for review when someone opens a pull request. # # We use a single team here, because if we use two teams, GitHub assigns two reviewers. -* @ZcashFoundation/general-rust-reviewers +* @ZcashFoundation/general-rust-reviewers + +# Frequently Modified Code +# +# This code is currently being modified in most PRs, +# so we assign reviews to the largest group of reviewers, +# to stop GitHub assigning multiple reviewers +# +# lightwalletd epic +/zebrad/src/commands/start.rs @ZcashFoundation/general-rust-reviewers # Network and Async Code /tower-batch/ @ZcashFoundation/network-reviewers /tower-fallback/ @ZcashFoundation/network-reviewers -/zebra-consensus/ @ZcashFoundation/network-reviewers /zebra-network/ @ZcashFoundation/network-reviewers /zebra-node-services/ @ZcashFoundation/network-reviewers -/zebra-state/ @ZcashFoundation/network-reviewers /zebra-tests/src/mock_service.rs @ZcashFoundation/network-reviewers /zebra-tests/src/service_extensions.rs @ZcashFoundation/network-reviewers /zebra-tests/src/transcript.rs @ZcashFoundation/network-reviewers -/zebra-utils/ @ZcashFoundation/network-reviewers -/zebrad/src/commands/ @ZcashFoundation/network-reviewers /zebrad/src/components/ @ZcashFoundation/network-reviewers # Cryptographic Code @@ -29,12 +34,12 @@ /zebra-chain/src/history_tree/ @ZcashFoundation/cryptographic-reviewers # Devops Code -/.github/ @ZcashFoundation/devops-reviewers +/.github/workflows/ @ZcashFoundation/devops-reviewers +/.github/mergify.yml @ZcashFoundation/devops-reviewers /docker/ @ZcashFoundation/devops-reviewers cloudbuild.yaml @ZcashFoundation/devops-reviewers codecov.yml @ZcashFoundation/devops-reviewers .dockerignore @ZcashFoundation/devops-reviewers -codecov.yml @ZcashFoundation/devops-reviewers firebase.json @ZcashFoundation/devops-reviewers katex-header.html @ZcashFoundation/devops-reviewers