Skip to content

Commit

Permalink
OWNERS set for video files in src (ampproject#25120)
Browse files Browse the repository at this point in the history
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 ampproject#24908).

Added missing ownership for other video/media related files in src.
  • Loading branch information
alanorozco authored and joshuarrrr committed Oct 22, 2019
1 parent 155d6a6 commit 2e2a965
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 1 deletion.
4 changes: 4 additions & 0 deletions src/OWNERS
Expand Up @@ -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'}],
},
],
}
11 changes: 11 additions & 0 deletions 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'}],
},
],
}
2 changes: 1 addition & 1 deletion src/service/video/OWNERS
Expand Up @@ -4,7 +4,7 @@
{
rules: [
{
owners: [{name: 'alanorozco'}, {name: 'wassgha'}],
owners: [{name: 'ampproject/wg-ui-and-a11y'}],
},
],
}

0 comments on commit 2e2a965

Please sign in to comment.