Skip to content
This repository has been archived by the owner on Oct 15, 2024. It is now read-only.

[FLOSS] Switch from probot/stale to github/stale #4476

Closed
kodebach opened this issue Sep 20, 2022 · 10 comments · Fixed by #4695 or #4726
Closed

[FLOSS] Switch from probot/stale to github/stale #4476

kodebach opened this issue Sep 20, 2022 · 10 comments · Fixed by #4695 or #4726
Labels

Comments

@kodebach
Copy link
Member

We should switch from the current Probot based "stale issue bot" to the one provided by GitHub.

Not only is the Probot based one, seemingly no longer maintained (see probot/stale#385 and probot/probot.github.io#376), but the GitHub one also has a lot more options. Including one option which allows closing issues as "not planned" instead of "completed".

@kodebach
Copy link
Member Author

@markus2330 The old Probot based bot also seems to be buggy. It closed both #3318 and #1610 even though the last comment wasn't the "I mark this issue stale" from the bot.

@markus2330 markus2330 added floss2022w 3p three points labels Sep 22, 2022
@stefnotch stefnotch self-assigned this Oct 18, 2022
@stefnotch stefnotch added the H1 label Oct 18, 2022
@stefnotch
Copy link
Contributor

I can confirm that probot/stale is still installed.
The config file exists at https://github.com/ElektraInitiative/libelektra/blob/master/.github/stale.yml
And since it's an app, it would also have to be uninstalled.

And the improved stale issues action still needs to be installed.

Installing the new stale issues action

After a fair bit of searching, I actually found the right documentation page
https://docs.github.com/en/actions/managing-issues-and-pull-requests/closing-inactive-issues

This can probably be done in a fork, and a pull request can be created with this.

Uninstalling probot/stale

By going to https://github.com/apps/stale and pressing configure.

Then one would select libelektra
image

And hit the x button
image

And finally press uninstall

@flo91 flo91 changed the title Switch from probot/stale to github/stale [FLOSS] Switch from probot/stale to github/stale Oct 18, 2022
@markus2330
Copy link
Contributor

Thank you for the detailed analysis. Let us first make a config file for the new one, then I'll do the steps to add the new one and remove the old one.

@stefnotch stefnotch removed their assignment Nov 20, 2022
@Janldeboer
Copy link
Contributor

Is creating the config file an appropriate change for a pull request in T1?
It does not completely fit into any of the 3 described problems, but it it is still closely related I suppose.

@markus2330
Copy link
Contributor

I uninstalled probot/stale as described above.

And the improved stale issues action still needs to be installed.

Is there anything else to be done?

@0x6178656c
Copy link
Contributor

And the improved stale issues action still needs to be installed.

Is there anything else to be done?

I don't think so. I think that the only manual step needed was removing the old probot/stale.

The action has been created automatically: https://github.com/ElektraInitiative/libelektra/actions/workflows/stale.yml

@kodebach
Copy link
Member Author

Is there a reason, we didn't end up using the close-issue-reason: not_planned option?

@markus2330
Copy link
Contributor

Is there a reason, we didn't end up using the close-issue-reason: not_planned option?

What do you mean? We switched.

@kodebach
Copy link
Member Author

Yes we switched to the GitHub stalebot action. But I don't see the close-issue-reason: not_planned option in https://github.com/ElektraInitiative/libelektra/blob/master/.github/workflows/stale.yml.

That means the bot still closes issues as "completed" not as "not planned". See for example #2991 (comment), which was closed today.

@markus2330 markus2330 reopened this Nov 29, 2022
@0x6178656c
Copy link
Contributor

0x6178656c commented Nov 29, 2022

@kodebach Thanks for pointing this out. I should have read this thread more carefully.

It seems that the option close-issue-reason is set automatically to not_planned in v6. I still added it so be explicit, though.

Edit: For context: I previously chose v5 for the GitHub action and have now changed it to v6 in #4726

@Janldeboer Janldeboer removed their assignment Nov 30, 2022
joni1993 pushed a commit to joni1993/libelektra that referenced this issue Dec 5, 2022
Janldeboer pushed a commit to Janldeboer/libelektra that referenced this issue Dec 12, 2022
Dynamichost96 pushed a commit to Dynamichost96/libelektra that referenced this issue Jan 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
5 participants