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

Leandro Lucarella via dmd-internals dmd-internals at puremagic.com
Sun May 8 15:15:08 PDT 2016


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


More information about the dmd-internals mailing list