Skip to content

Commit

Permalink
Remove unnecessary owners in client_channel tree.
Browse files Browse the repository at this point in the history
  • Loading branch information
markdroth committed Nov 15, 2019
1 parent 9273356 commit 80c69d8
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 15 deletions.
2 changes: 1 addition & 1 deletion .github/CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -4,5 +4,5 @@
/**/OWNERS @markdroth @nicolasnoble @a11r
/bazel/** @nicolasnoble @jtattermusch @veblush @gnossen
/cmake/** @jtattermusch @nicolasnoble @apolcyn
/src/core/ext/filters/client_channel/** @markdroth @apolcyn @AspirinSJL
/src/core/ext/filters/client_channel/** @markdroth
/tools/dockerfile/** @jtattermusch @apolcyn @nicolasnoble
12 changes: 0 additions & 12 deletions OWNERS
Original file line number Diff line number Diff line change
@@ -1,16 +1,4 @@
# Top level ownership

# nothing listed here until GitHub CODEOWNERS gets better
# we need:
# 1. owners to be able to self-approve
# 2. authors to be able to select approvers

# OWNERS file approvers
# POLICY: at least three owners are needed before adding any OWNERS
# REASON: GitHub does not recognize an author as able to give approval
# for a change; without this policy authors that are owners would
# be forced to rely on one reviewer, which would consequently
# lead to a bus factor of one to changes to that code
@markdroth **/OWNERS
@nicolasnoble **/OWNERS
@a11r **/OWNERS
Expand Down
2 changes: 0 additions & 2 deletions src/core/ext/filters/client_channel/OWNERS
Original file line number Diff line number Diff line change
@@ -1,4 +1,2 @@
set noparent
@markdroth
@apolcyn
@AspirinSJL

0 comments on commit 80c69d8

Please sign in to comment.