From 731e3cef0c200bcf06e8d317503582f62f64cbc4 Mon Sep 17 00:00:00 2001 From: Billy Lynch Date: Tue, 10 Oct 2023 10:57:13 -0400 Subject: [PATCH] Add reviewers section to OWNERS file. This gives more obvious place to add people as reviewers to lgtm PRs without needing to grant them full approver. Removes ImJasonH, lbernick, wlynch from active reviewers. Removes productivity-reviewers in favor of using the current approvers. --- OWNERS | 3 +++ OWNERS_ALIASES | 14 ++++++++++---- hack/OWNERS | 2 +- 3 files changed, 14 insertions(+), 5 deletions(-) diff --git a/OWNERS b/OWNERS index b36e9078c3b..ca09ccbebec 100644 --- a/OWNERS +++ b/OWNERS @@ -2,3 +2,6 @@ approvers: - pipeline-approvers + +reviewers: +- pipeline-reviewers \ No newline at end of file diff --git a/OWNERS_ALIASES b/OWNERS_ALIASES index df666b7b228..306af8da639 100644 --- a/OWNERS_ALIASES +++ b/OWNERS_ALIASES @@ -11,6 +11,16 @@ aliases: - abayer - wlynch + pipeline-reviewers: + - afrittoli + - bobcatfish + - dibyom + - vdemeester + - pritidesai + - jerop + - abayer + # - wlynch (feel free to /cc me into PRs as needed) + apis-approvers: - afrittoli - bobcatfish @@ -27,10 +37,6 @@ aliases: - afrittoli - bobcatfish - vdemeester - productivity-reviewers: - - dibyom - - dlorenc - - ImJasonH # Alumni ❤️ # tejal29 diff --git a/hack/OWNERS b/hack/OWNERS index 1b3eb9f57db..1952d93629e 100644 --- a/hack/OWNERS +++ b/hack/OWNERS @@ -1,6 +1,6 @@ approvers: - productivity-approvers reviewers: -- productivity-reviewers +- productivity-approvers labels: - area/test-infra \ No newline at end of file