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.

Sign up for GitHub

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

Jump to bottom

[owners] Notify required reviewers #1012

Closed
rcebulko opened this issue Nov 10, 2020 · 0 comments · Fixed by #1013
Closed

[owners] Notify required reviewers #1012

rcebulko opened this issue Nov 10, 2020 · 0 comments · Fixed by #1013
Assignees
@rcebulko
Labels
Category: Owners P1: High Priority Type: Feature Request

Comments

@rcebulko
Copy link
Contributor

Currently required reviewers block owners approval, but may not be notified about PRs touching the code they own. By default, PRs with required reviewers should include them in notification comments.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rcebulko rcebulko

Labels
Category: Owners P1: High Priority Type: Feature Request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[owners] Notify owners with required: true rcebulko/amp-github-apps
1 participant
@rcebulko