From 2e2a965d1c2b830181a44c9959d44529d8ff2d4f Mon Sep 17 00:00:00 2001 From: Alan Orozco Date: Fri, 18 Oct 2019 08:44:46 -0700 Subject: [PATCH] OWNERS set for video files in src (#25120) Realized that video service files are really touched when creating 3p player contributions, so they can be set to the broader ampproject/wg-ui-and-a11y group (rather than the specific set from #24908). Added missing ownership for other video/media related files in src. --- src/OWNERS | 4 ++++ src/service/OWNERS | 11 +++++++++++ src/service/video/OWNERS | 2 +- 3 files changed, 16 insertions(+), 1 deletion(-) create mode 100644 src/service/OWNERS diff --git a/src/OWNERS b/src/OWNERS index 4c4407e13ff03..82e99434a7f64 100644 --- a/src/OWNERS +++ b/src/OWNERS @@ -6,5 +6,9 @@ { owners: [{name: 'dvoytenko'}, {name: 'ampproject/wg-runtime'}], }, + { + pattern: '{iframe-video,mediasession-helper,video-*}.js', + owners: [{name: 'ampproject/wg-ui-and-a11y'}], + }, ], } diff --git a/src/service/OWNERS b/src/service/OWNERS new file mode 100644 index 0000000000000..36f6a5efb4aa3 --- /dev/null +++ b/src/service/OWNERS @@ -0,0 +1,11 @@ +// For an explanation of the OWNERS rules and syntax, see: +// https://github.com/ampproject/amp-github-apps/blob/master/owners/OWNERS.example + +{ + rules: [ + { + pattern: 'video-*.js', + owners: [{name: 'ampproject/wg-ui-and-a11y'}], + }, + ], +} diff --git a/src/service/video/OWNERS b/src/service/video/OWNERS index b472fae881731..6b1d001d15c8c 100644 --- a/src/service/video/OWNERS +++ b/src/service/video/OWNERS @@ -4,7 +4,7 @@ { rules: [ { - owners: [{name: 'alanorozco'}, {name: 'wassgha'}], + owners: [{name: 'ampproject/wg-ui-and-a11y'}], }, ], }