Update OWNERS - #211
Conversation
Add GKE team members as reviewers
There was a problem hiding this comment.
Code Review
This pull request updates the OWNERS file by adding a new reviewers section. The review feedback suggests removing redundant entries for individuals who are already listed as approvers and recommends sorting the list alphabetically to improve maintainability.
| - bradhoekstra | ||
| - GinnyJI | ||
| - stalhaali | ||
| - toshiowang | ||
| - jayantid | ||
| - dshnayder |
There was a problem hiding this comment.
The reviewers list contains redundant entries and is not alphabetically sorted. bradhoekstra and GinnyJI are already listed as approvers (lines 2 and 6). In standard OWNERS file implementations (such as those used in Kubernetes-related projects), approvers are implicitly reviewers, so they do not need to be duplicated in this section. Additionally, sorting the list alphabetically improves maintainability and helps prevent future duplicates.
- dshnayder
- jayantid
- stalhaali
- toshiowang
|
/verify-owners |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bradhoekstra, erain The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/verify-owners |
Add GKE team members as reviewers