I'm +0 on the reviewer bot pings. Closing PRs where the author hasn't updated in 30 days is something a maintainer would have to do anyways, so it seems like a useful automation. And there's only one author, so it's guaranteed to ping the right person. Things are not so clean with reviewers.
With the labels and codeowners file [1] I think we have supplied sufficient tools so that each subproject in the monorepo can manage their review process in their own way. For example, I have a bookmark that takes me to a filtered view of PRs that only shows me the C++ Parquet ones that are ready for review [2]. I'd encourage each reviewer to have a similar view of the project that they regularly check. [1] https://github.com/apache/arrow/blob/main/.github/CODEOWNERS [2] https://github.com/apache/arrow/pulls?q=is%3Aopen+is%3Apr+label%3A%22Component%3A+C%2B%2B%22+label%3A%22Component%3A+Parquet%22+draft%3Afalse On Thu, Mar 30, 2023 at 1:37 PM Anja <anja.kef...@gmail.com> wrote: > Using those labels is a clever idea! > > Would there be a benefit to pinging reviewers for PRs that have been > "awaiting X review" for more than 30 days? > > On Thu, 30 Mar 2023 at 12:31, Will Jones <will.jones...@gmail.com> wrote: > > > Thanks Raul. Perhaps we could limit the stale bot to PRs that have been > in > > "awaiting changes" for 30 or more days? > > > > On Thu, Mar 30, 2023 at 11:36 AM Raúl Cumplido <raulcumpl...@gmail.com> > > wrote: > > > > > I suppose we could use the new labels for "awaiting review", "awaiting > > > committer review", "awaiting changes" and "awaiting change review" to > > know > > > whether is stale due to the contributor or the reviewer. > > > > > > El jue, 30 mar 2023, 20:08, Will Jones <will.jones...@gmail.com> > > escribió: > > > > > > > First, to clarify: we are discussing for the monorepo only, not for > > Rust > > > / > > > > Julia / etc.? This is a big project, so best to be specific which > > > > subprojects you are addressing. > > > > > > > > I am +0.5 on this. 30 days seems like an appropriate window for this > > > > project. If the PR was stale because the contributor had not updated > > it, > > > it > > > > seems appropriate. But sometimes it's because it hasn't had an update > > > from > > > > reviewers for a while, and in that situation it doesn't seem as > ideal. > > > > > > > > > > > > > > > > On Thu, Mar 30, 2023 at 11:01 AM Anja <anja.kef...@gmail.com> wrote: > > > > > > > > > Also, perhaps it can be two bots in an escalated process. A > "reminder > > > > ping" > > > > > bot every X days, and then a stalebot every X+Y days. > > > > > > > > > > On Thu, 30 Mar 2023 at 10:54, Anja <anja.kef...@gmail.com> wrote: > > > > > > > > > > > When checked this morning, there were 119 PRs that haven't been > > > updated > > > > > in > > > > > > 30 days. The oldest was nearly 3 years old. > > > > > > > > > > > > I propose the addition of a bot that will automatically close any > > PRs > > > > > that > > > > > > haven't been updated in 30 days. The closing will act as a > > > notification > > > > > to > > > > > > the reviewers and submitter to evaluate if the work still has > > value, > > > > and > > > > > > just outright close work that is too out-dated for a > > straightforward > > > > > merge. > > > > > > > > > > > > If the behaviour is done by a bot, it could reduce maintenance > > > burden, > > > > > and > > > > > > simplify the emotional response. A bot can link to a policy, and > it > > > > feels > > > > > > neutral in its consistent tone. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
