Skip to content

Commit

Permalink
🏗 Convert all OWNERS.yaml files to JSON5 format (#24934)
Browse files Browse the repository at this point in the history
  • Loading branch information
rcebulko authored and rsimha committed Oct 7, 2019
1 parent d844f99 commit a5785e6
Show file tree
Hide file tree
Showing 301 changed files with 1,995 additions and 726 deletions.
12 changes: 12 additions & 0 deletions 3p/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
// For an explanation of the OWNERS rules and syntax, see:
// https://github.com/ampproject/amp-github-apps/blob/master/owners/OWNERS.example

{
rules: [
{
owners: [
{ name: 'ampproject/wg-ads' }
]
}
]
}
4 changes: 0 additions & 4 deletions 3p/OWNERS.yaml

This file was deleted.

79 changes: 79 additions & 0 deletions OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,79 @@
// For an explanation of the OWNERS rules and syntax, see:
// https://github.com/ampproject/amp-github-apps/blob/master/owners/OWNERS.example

{
rules: [
{
owners: [
{
name: 'cramforce',
requestReviews: false
},
{
name: 'dvoytenko',
requestReviews: false
},
{
name: 'jridgewell',
requestReviews: false
}
]
},

{
pattern: '**/validator-*.{protoascii,html,out}',
owners: [
{
name: 'ampproject/wg-caching',
notify: true
}
]
},

{
pattern: '*.md',
owners: [
{ name: 'ampproject/wg-outreach' }
]
},

{
pattern: '{.*,babel.config.js,gulpfile.js}',
owners: [
{ name: 'ampproject/wg-infra' }
]
},

{
pattern: '{package.json,yarn.lock}',
owners: [
{ name: 'ampproject/wg-infra' },
{ name: 'ampproject/wg-runtime' },
{ name: 'ampproject/wg-performance' }
]
},

{
// The below rule will be here only temporarily during the transition
// period while many owners files are being updated as contributors and
// teams notice which files are out of date. The "notify" option means the
// owners bot will tag `rcebulko` on any PR that touches an owners file.
// This will allow verification of syntax and semantics for all updates
// until the syntax check API for Travis to use is complete.

// Though this gives `rcebulko` the ability to approve changes to owners
// files, he does not currently have merge access and he runs and
// maintains the owners bot application.

// TODO(#24685): Remove this once most owners files have been updated
// and/or the Travis check is implemented.
pattern: '**/{OWNERS.yaml,OWNERS}',
owners: [
{
name: 'rcebulko',
notify: true
}
]
}
]
}
28 changes: 0 additions & 28 deletions OWNERS.yaml

This file was deleted.

12 changes: 12 additions & 0 deletions ads/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
// For an explanation of the OWNERS rules and syntax, see:
// https://github.com/ampproject/amp-github-apps/blob/master/owners/OWNERS.example

{
rules: [
{
owners: [
{ name: 'ampproject/wg-ads' }
]
}
]
}
4 changes: 0 additions & 4 deletions ads/OWNERS.yaml

This file was deleted.

12 changes: 12 additions & 0 deletions ads/google/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
// For an explanation of the OWNERS rules and syntax, see:
// https://github.com/ampproject/amp-github-apps/blob/master/owners/OWNERS.example

{
rules: [
{
owners: [
{ name: 'ampproject/a4a' }
]
}
]
}
4 changes: 0 additions & 4 deletions ads/google/OWNERS.yaml

This file was deleted.

12 changes: 12 additions & 0 deletions build-system/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
// For an explanation of the OWNERS rules and syntax, see:
// https://github.com/ampproject/amp-github-apps/blob/master/owners/OWNERS.example

{
rules: [
{
owners: [
{ name: 'ampproject/wg-infra' }
]
}
]
}
4 changes: 0 additions & 4 deletions build-system/OWNERS.yaml

This file was deleted.

15 changes: 15 additions & 0 deletions build-system/global-configs/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
// For an explanation of the OWNERS rules and syntax, see:
// https://github.com/ampproject/amp-github-apps/blob/master/owners/OWNERS.example

{
rules: [
{
owners: [
{
name: 'ampproject/wg-approvers',
requestReviews: false
}
]
}
]
}
5 changes: 0 additions & 5 deletions build-system/global-configs/OWNERS.yaml

This file was deleted.

12 changes: 12 additions & 0 deletions contributing/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
// For an explanation of the OWNERS rules and syntax, see:
// https://github.com/ampproject/amp-github-apps/blob/master/owners/OWNERS.example

{
rules: [
{
owners: [
{ name: 'mrjoro' }
]
}
]
}
4 changes: 0 additions & 4 deletions contributing/OWNERS.yaml

This file was deleted.

13 changes: 13 additions & 0 deletions css/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
// For an explanation of the OWNERS rules and syntax, see:
// https://github.com/ampproject/amp-github-apps/blob/master/owners/OWNERS.example

{
rules: [
{
owners: [
{ name: 'aghassemi' },
{ name: 'camelburrito' }
]
}
]
}
5 changes: 0 additions & 5 deletions css/OWNERS.yaml

This file was deleted.

12 changes: 12 additions & 0 deletions examples/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
// For an explanation of the OWNERS rules and syntax, see:
// https://github.com/ampproject/amp-github-apps/blob/master/owners/OWNERS.example

{
rules: [
{
owners: [
{ name: '*' }
]
}
]
}
4 changes: 0 additions & 4 deletions examples/OWNERS.yaml

This file was deleted.

15 changes: 15 additions & 0 deletions examples/visual-tests/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
// For an explanation of the OWNERS rules and syntax, see:
// https://github.com/ampproject/amp-github-apps/blob/master/owners/OWNERS.example

{
rules: [
{
owners: [
{ name: 'danielrozenberg' },
{ name: 'erwinmombay' },
{ name: 'estherkim' },
{ name: 'rsimha' }
]
}
]
}
7 changes: 0 additions & 7 deletions examples/visual-tests/OWNERS.yaml

This file was deleted.

25 changes: 25 additions & 0 deletions extensions/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
// For an explanation of the OWNERS rules and syntax, see:
// https://github.com/ampproject/amp-github-apps/blob/master/owners/OWNERS.example

{
rules: [
{
owners: [
{ name: 'aghassemi' }
]

This comment has been minimized.

Copy link
@OtariaTk

OtariaTk Feb 18, 2024

Foda se viado

},
{
pattern: '**/*-player.{js,md}',
owners: [
{
name: 'alanorozco',
notify: true
},
{
name: 'wassgha',
notify: true
}
]
}
]
}
7 changes: 0 additions & 7 deletions extensions/OWNERS.yaml

This file was deleted.

13 changes: 13 additions & 0 deletions extensions/amp-3d-gltf/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
// For an explanation of the OWNERS rules and syntax, see:
// https://github.com/ampproject/amp-github-apps/blob/master/owners/OWNERS.example

{
rules: [
{
owners: [
{ name: 'mixtur' },
{ name: 'ampproject/wg-ui-and-a11y' }
]
}
]
}
5 changes: 0 additions & 5 deletions extensions/amp-3d-gltf/OWNERS.yaml

This file was deleted.

12 changes: 12 additions & 0 deletions extensions/amp-3q-player/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
// For an explanation of the OWNERS rules and syntax, see:
// https://github.com/ampproject/amp-github-apps/blob/master/owners/OWNERS.example

{
rules: [
{
owners: [
{ name: 'ampproject/wg-ui-and-a11y' }
]
}
]
}
4 changes: 0 additions & 4 deletions extensions/amp-3q-player/OWNERS.yaml

This file was deleted.

12 changes: 12 additions & 0 deletions extensions/amp-a4a/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
// For an explanation of the OWNERS rules and syntax, see:
// https://github.com/ampproject/amp-github-apps/blob/master/owners/OWNERS.example

{
rules: [
{
owners: [
{ name: 'ampproject/a4a' }
]
}
]
}
5 changes: 0 additions & 5 deletions extensions/amp-a4a/OWNERS.yaml

This file was deleted.

14 changes: 14 additions & 0 deletions extensions/amp-access-scroll/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
// For an explanation of the OWNERS rules and syntax, see:
// https://github.com/ampproject/amp-github-apps/blob/master/owners/OWNERS.example

{
rules: [
{
owners: [
{ name: 'kushal' },
{ name: 'dbow' },
{ name: 'junoatscroll' }
]
}
]
}
6 changes: 0 additions & 6 deletions extensions/amp-access-scroll/OWNERS.yaml

This file was deleted.

0 comments on commit a5785e6

Please sign in to comment.