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.
JeromeJu · Oct 18, 2023 · e07e5e3 · e07e5e3
1 parent c7de7b1
commit e07e5e3
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 5 deletions.
diff --git a/OWNERS b/OWNERS
@@ -2,3 +2,6 @@
 
 approvers:
 - pipeline-approvers
+
+reviewers:
+- pipeline-reviewers
diff --git a/OWNERS_ALIASES b/OWNERS_ALIASES
@@ -13,6 +13,16 @@ aliases:
   - yongxuanzhang
   - chitrangpatel
 
+  pipeline-reviewers:
+  - afrittoli
+  - bobcatfish
+  - dibyom
+  - vdemeester
+  - pritidesai
+  - jerop
+  - abayer
+  # - wlynch (feel free to /cc me into PRs as needed)
+
   apis-approvers:
   - afrittoli
   - bobcatfish
@@ -29,10 +39,6 @@ aliases:
   - afrittoli
   - bobcatfish
   - vdemeester
-  productivity-reviewers:
-  - dibyom
-  - dlorenc
-  - ImJasonH
 
 # Alumni ❤️
 # tejal29

diff --git a/hack/OWNERS b/hack/OWNERS
@@ -1,6 +1,6 @@
 approvers:
 - productivity-approvers
 reviewers:
-- productivity-reviewers
+- productivity-approvers
 labels:
 - area/test-infra