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

Assign maintainers in addition to requesting review #93

Open
adamjstewart opened this issue Nov 18, 2023 · 2 comments
Open

Assign maintainers in addition to requesting review #93

adamjstewart opened this issue Nov 18, 2023 · 2 comments

Comments

@adamjstewart
Copy link
Member

Spackbot automatically requests reviews from package maintainers. However, we've also been assigning people to PRs to make it easier to visualize which PRs need attention. We should teach spackbot how to do this too. Assigning may require a higher level of permissions than requesting review, so this may need a try-except.

@alecbcs
Copy link
Member

alecbcs commented Nov 18, 2023

Yep this is a great idea. Would you suggest we assign all maintainers or only those with merge-to-develop permissions?

I've thought about implementing this feature before, but with the constraint of only assigning folks who have triage permissions.

@adamjstewart
Copy link
Member Author

My vote would be only people with merge-to-develop permissions

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

No branches or pull requests

2 participants