.github/workflows: remove auto-requested reviewers#42952
Conversation
If reviews were requested by renovate or the auto-committer it means it happened on a auto-approve PR and therefore we don't need to request reviews from anyone. Signed-off-by: André Martins <andre@cilium.io>
|
/test |
|
We've hit some problems recently where renovate has been bumping dependencies we're not ready to adopt, for instance bumping LLVM major versions on stable branches. Are we ready to fully decouple reviews from humans in this area now? We might also want to think about formalizing the triage process for these PRs if no-one is auto-assigned. I occasionally take a look at renovate PRs but it's not super obvious to me what happens if the PR gets stuck on CI for instance. |
Oh, I didn't know this. Then it should be a matter of updating renovate configuration to take into account those constrains.
The reason for automating PRs is that we don't need to look at them. Me and Julian usually check the PRs on a weekly basis and see they are getting stuck for more than a week. If that happens I ping directly the owners of that dependency. |
If reviews were requested by renovate or the auto-committer it means it happened on a auto-approve PR and therefore we don't need to request reviews from anyone.