Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🏗 Add OWNERS files for all build-system/ sub-directories #24945

Merged
merged 1 commit into from Oct 8, 2019
Merged

🏗 Add OWNERS files for all build-system/ sub-directories #24945

merged 1 commit into from Oct 8, 2019

Conversation

rsimha
Copy link
Contributor

@rsimha rsimha commented Oct 7, 2019

This PR adds OWNERS files across build-system/.

/cc @ampproject/wg-infra @ampproject/wg-runtime @ampproject/wg-performance since there's some common code that everyone must be able to edit.

@rsimha
Copy link
Contributor Author

rsimha commented Oct 7, 2019

I've populated all these OWNERS files with an initial guess for who needs to be notified. Happy to change any of these on request.

@estherkim
Copy link
Collaborator

I like mine, thanks!

Copy link
Contributor

@rcebulko rcebulko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the strategy of adding a team and notifying specific users!

@rsimha rsimha merged commit 33dc62a into ampproject:master Oct 8, 2019
@rsimha rsimha deleted the 2019-10-07-BuildSystemOwners branch October 8, 2019 13:53
},
{
pattern: 'internal-version.js',
owners: [{ name: 'danielrozenberg', notify: true }]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You edit a file ONCE… 😅

{
owners: [
{ name: 'ampproject/wg-infra' },
{ name: 'rsimha', notify: true }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can add me here as a notify too

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already merged. Can you add in a new PR?

@amp-owners-bot
Copy link

amp-owners-bot bot commented Oct 8, 2019

Hey @alanorozco, these files were changed:

  • build-system/server/app-index/OWNERS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants