Assign maintainers in addition to requesting review
adamjstewart opened this issue · 2 comments
adamjstewart commented
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 commented
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 commented
My vote would be only people with merge-to-develop permissions