search

spack / spackbot

Spack maintainer bot 🤖
https://spack.github.io/spackbot/
Other
8 stars 12 forks source link

Assign maintainers in addition to requesting review #93

Open adamjstewart opened 11 months ago

adamjstewart commented 11 months ago

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 11 months ago

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 11 months ago

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