weekly-digest icon indicating copy to clipboard operation
weekly-digest copied to clipboard

Feature: Auto close old issues labelled weekly-digest

Open abhijeetps opened this issue 7 years ago • 6 comments

I think that the automatic creation of Weekly Digests as an issue might become annoying if there are too many Weekly Digests in a repository and the old Weekly Digests aren't closed.

What I propose is that immediately before the bot creates a new Weekly Digests, it closes the old Weekly Digest(s) having weekly-digest label if they exist.

I am thinking of enabling it by default and will provide user option to disable it. Let me know what do you think.

cc @gr2m , @wilhelmklopp

abhijeetps avatar Jul 13 '18 14:07 abhijeetps

Sounds great! Makes a lot of sense to me to close these issues :)

wilhelmklopp avatar Jul 13 '18 15:07 wilhelmklopp

@aps120797 nice idea, one thing though.

Suppose while opening the weekly digest issue, there is an error and so the issue is not created, but the app would have already closed the last weekly digest issue by then. Why not close the last weekly digest issue only after the new weekly digest issue is opened?

itaditya avatar Jul 13 '18 19:07 itaditya

what about using one issue and adding comments so everything is easily viewed in one place?

steveoh avatar Aug 30 '18 16:08 steveoh

@steveoh I think that's the right idea, too. We brought this up here: https://github.com/nock/nock/issues/1211#issuecomment-420350043. Linking to keep track of this feature.

RichardLitt avatar Sep 12 '18 13:09 RichardLitt

+1 to this feature. It would be great if old issues are closed automatically

MissingRoberto avatar Sep 28 '18 19:09 MissingRoberto

@abhijeetps I would like to help in the addition of this feature is it still open ?

sladyn98 avatar Nov 15 '19 12:11 sladyn98