Merge pull request #11800 from ctiller/owners

Use ** for OWNERS files for trees
pull/11798/merge
Nicolas Noble 7 years ago committed by GitHub
commit 1a984a5590
  1. 34
      .github/CODEOWNERS
  2. 4
      tools/mkowners/mkowners.py

@ -1,22 +1,22 @@
# Auto-generated by the tools/mkowners/mkowners.py tool # Auto-generated by the tools/mkowners/mkowners.py tool
# Uses OWNERS files in different modules throughout the # Uses OWNERS files in different modules throughout the
# repository as the source of truth for module ownership. # repository as the source of truth for module ownership.
* @a11r @nicolasnoble @ctiller ** @a11r @nicolasnoble @ctiller
bazel/* @nicolasnoble @dgquintas @ctiller bazel/** @nicolasnoble @dgquintas @ctiller
cmake/* @jtattermusch @a11r @nicolasnoble @ctiller cmake/** @jtattermusch @a11r @nicolasnoble @ctiller
doc/PROTOCOL-HTTP2.md @ejona86 @a11r @nicolasnoble @ctiller doc/PROTOCOL-HTTP2.md @ejona86 @a11r @nicolasnoble @ctiller
doc/interop-test-descriptions.md @ejona86 @a11r @nicolasnoble @ctiller doc/interop-test-descriptions.md @ejona86 @a11r @nicolasnoble @ctiller
etc/* @jboeuf @nicolasnoble @a11r @ctiller etc/** @jboeuf @nicolasnoble @a11r @ctiller
include/* @ctiller @markdroth @dgquintas @a11r @nicolasnoble include/** @ctiller @markdroth @dgquintas @a11r @nicolasnoble
src/core/* @ctiller @markdroth @dgquintas @a11r @nicolasnoble src/core/** @ctiller @markdroth @dgquintas @a11r @nicolasnoble
src/cpp/* @ctiller @markdroth @dgquintas @a11r @nicolasnoble src/cpp/** @ctiller @markdroth @dgquintas @a11r @nicolasnoble
src/csharp/* @jtattermusch @apolcyn @a11r @nicolasnoble @ctiller src/csharp/** @jtattermusch @apolcyn @a11r @nicolasnoble @ctiller
src/node/* @murgatroid99 @a11r @nicolasnoble @ctiller src/node/** @murgatroid99 @a11r @nicolasnoble @ctiller
src/objective-c/* @muxi @makdharma @a11r @nicolasnoble @ctiller src/objective-c/** @muxi @makdharma @a11r @nicolasnoble @ctiller
src/php/* @stanley-cheung @murgatroid99 @a11r @nicolasnoble @ctiller src/php/** @stanley-cheung @murgatroid99 @a11r @nicolasnoble @ctiller
src/python/* @nathanielmanistaatgoogle @kpayson64 @a11r @nicolasnoble @ctiller src/python/** @nathanielmanistaatgoogle @kpayson64 @a11r @nicolasnoble @ctiller
src/ruby/* @apolcyn @murgatroid99 @a11r @nicolasnoble @ctiller src/ruby/** @apolcyn @murgatroid99 @a11r @nicolasnoble @ctiller
tools/* @matt-kwong @jtattermusch @nicolasnoble @a11r @ctiller tools/** @matt-kwong @jtattermusch @nicolasnoble @a11r @ctiller
tools/codegen/core/* @ctiller @dgquintas @markdroth tools/codegen/core/** @ctiller @dgquintas @markdroth
tools/dockerfile/* @matt-kwong @jtattermusch @nicolasnoble @a11r @ctiller tools/dockerfile/** @matt-kwong @jtattermusch @nicolasnoble @a11r @ctiller
tools/run_tests/* @matt-kwong @jtattermusch @nicolasnoble @a11r @ctiller tools/run_tests/** @matt-kwong @jtattermusch @nicolasnoble @a11r @ctiller

@ -141,7 +141,7 @@ def add_parent_to_globs(parent, globs):
for owners in owners_data: for owners in owners_data:
if owners.dir == parent: if owners.dir == parent:
for directive in owners.directives: for directive in owners.directives:
for dglob in directive.globs or ['*']: for dglob in directive.globs or ['**']:
for gglob, glob in globs.items(): for gglob, glob in globs.items():
if glob_intersect(dglob, gglob): if glob_intersect(dglob, gglob):
if directive.who not in glob: if directive.who not in glob:
@ -163,7 +163,7 @@ with open(args.out, 'w') as out:
continue continue
globs = collections.OrderedDict() globs = collections.OrderedDict()
for directive in head.directives: for directive in head.directives:
for glob in directive.globs or ['*']: for glob in directive.globs or ['**']:
if glob not in globs: if glob not in globs:
globs[glob] = [] globs[glob] = []
globs[glob].append(directive.who) globs[glob].append(directive.who)

Loading…
Cancel
Save