[dmd-internals] Auto-warn authors followed by auto-close of very stale PRs

[Andrej Mitrovic via dmd-internals]

There's also the PR effect (public relations, not pull request). If
people see there's hundreds of open PRs they might reach bad
conclusions, like "my code will probably never be merged so I better
not even try contributing".

On 5/9/16, Leandro Lucarella via dmd-internals
<dmd-internals at puremagic.com> wrote:
> BTW, having old broken PRs consumes resources in the auto-tester, right?
> Once a PR have rotten it will be never magically fixed, so it doesn't
> make sense to keep testing it with the auto-tester.
>
> Leandro Lucarella via dmd-internals, el  9 de May a las 00:15 me escribiste:
>> If a PR is extremely old and have rotten for ages, it's not really
>> useful, so is just noise in the PR list. I think if you are conservative
>> about the "X amount of time" it might help. You can also always come to
>> old closed PRs (you could use a label to mark PR automatically closed).
>>
>> Also, if you want a PR to stay alive, you just have to tell the bot
>> that. I guess the bot wouldn't close the PR if it has any activity after
>> the warning...
>>
>> Walter Bright via dmd-internals, el  8 de May a las 07:55 me escribiste:
>> > I agree with Daniel. Closing PRs just because they are old has no
>> > benefit, and sometimes we do go back, fix one, and pull it, or use
>> > it as a guide to making a better PR.
>> >
>> >
>> > On 5/7/2016 9:35 AM, Daniel Murphy via dmd-internals wrote:
>> > >What would that gain us?  I eventually come back to my open PRs...
>> > >
>> > >On Sat, May 7, 2016 at 12:11 AM, Andrej Mitrovic via dmd-internals
>> > ><dmd-internals at puremagic.com> wrote:
>> > >>It seems that after open pulls page 4 or so on the DMD side of things
>> > >>a pretty large number of PRs are completely broken and haven't been
>> > >>touched in months, or worse in years. E.g.
>> > >>https://github.com/dlang/dmd/pull/4658
>> > >>
>> > >>There is really no benefit to having PRs just sit idly like that.
>> > >>
>> > >>I think we could automate the clean-up of stale pulls and we could
>> > >>even do it in stages:
>> > >>
>> > >>After X amount of time we could make `dlang-bot` attempt to notify the
>> > >>author of his stale PR, then let some more time pass again, and if the
>> > >>PR is still in bad shape `dlang-bot` would close the PR, send a
>> > >>notification to the PR author about the pull being closed, but also CC
>> > >>to dmd-internals / phobos-internals / etc in case the author went AWOL
>> > >>so we have a chance of picking up the work from there.
>> > >>
>> > >>Thoughts?
>> > >>_______________________________________________
>> > >>dmd-internals mailing list
>> > >>dmd-internals at puremagic.com
>> > >>http://lists.puremagic.com/mailman/listinfo/dmd-internals
>> > >_______________________________________________
>> > >dmd-internals mailing list
>> > >dmd-internals at puremagic.com
>> > >http://lists.puremagic.com/mailman/listinfo/dmd-internals
>> >
>> > _______________________________________________
>> > dmd-internals mailing list
>> > dmd-internals at puremagic.com
>> > http://lists.puremagic.com/mailman/listinfo/dmd-internals
>>
>> --
>> Leandro Lucarella (AKA luca)                     http://llucax.com.ar/
>> ----------------------------------------------------------------------
>> They love me like I was a brother
>> They protect me, listen to me
>> They dug me my very own garden
>> Gave me sunshine, made me happy
>>
>> Nice dream, nice dream
>> Nice dream
>> _______________________________________________
>> dmd-internals mailing list
>> dmd-internals at puremagic.com
>> http://lists.puremagic.com/mailman/listinfo/dmd-internals
>
> --
> Leandro Lucarella (AKA luca)                     http://llucax.com.ar/
> ----------------------------------------------------------------------
> Cada movimiento que no se hace, es un movimiento que se pierde.
> Y cada movimiento que se pierde, se transforma en una mochila.
> Y las mochilas nos alejan, de nuestros amigos y nuestras amigas.
> Y nuestros amigos se transforman, en enemigos y en enemigas.
> Cada movimiento que se hace, es una mochila que se deja.
> _______________________________________________
> dmd-internals mailing list
> dmd-internals at puremagic.com
> http://lists.puremagic.com/mailman/listinfo/dmd-internals