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
Set @alanorozco and @wassgha as owners of video service #24908
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aside from the comment, LGTM from a syntax perspective. Please get owners approval from an owner of src
or one of the subdirectories; I'm only an owner for the purposes of monitoring owners syntax and best practices, but do not have authority to approve actual changes in ownership.
JSON format
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might consider using the alias for the wg
instead in the future.
@kristoferbaxter I'm consistently setting this OWNER set for video-related files as not to spam the rest of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm
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.
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.
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.
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.
No description provided.