dlang-bot icon indicating copy to clipboard operation
dlang-bot copied to clipboard

Published 20 hours ago • verified publisher icon dlang

Send a warning about old open PRs targeting stable

Open wilzbach opened this issue 8 years ago • 2 comments
trafficstars

http://forum.dlang.org/post/[email protected]

  • Maybe two days instead of one?
  • Mail to Dlang-internal?

wilzbach avatar Jul 01 '17 18:07 wilzbach

To whom? I'm checking stable PR during betas, using a tinsy bash script. https://github.com/MartinNowak/github_scripts/blob/62c7b0197f69d482e5dfbca55a950976fc681f09/list_branch_pulls.sh Seems easier to do as part of the release script that to automate atm.

MartinNowak avatar Jan 15 '18 17:01 MartinNowak

I check them at auto-tester: https://auto-tester.puremagic.com/pulls.ghtml?projectid=14

BTW the bot already recognizes stalled PRs targetting stable (see e.g. https://github.com/dlang/phobos/pull/5981), so it would be just a matter of instead of labelling, commenting with sth. like:

Hey everyone, this PR is targeting stable and hasn't received any activity in the last three days. PRs to stable should be simple bug fixes or receive a higher priority. @: If you need help to finish this PR, join # on Slack.

wilzbach avatar Jan 15 '18 22:01 wilzbach