Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stale issue/pr action #4625

Merged
merged 8 commits into from
Mar 8, 2021
Merged

Stale issue/pr action #4625

merged 8 commits into from
Mar 8, 2021

Conversation

snide
Copy link
Contributor

@snide snide commented Mar 8, 2021

Summary

Adds a Github action that checks for stale issues and PRs using https://github.com/actions/stale, which is Github's officially supported action for stale checks. To run this in a dry run mode, I've turned on Github logging for the repo, and set debug-only: true for the action. The docs on https://github.com/actions/stale#debugging suggest this means the action will not be run, and only be logged. This should allow us to watch how this would effect our repo.

Checklist

@snide snide added the skip-changelog Use on PRs to skip changelog requirement (Don't delete - used for automation) label Mar 8, 2021
@cchaos
Copy link
Contributor

cchaos commented Mar 8, 2021

Where do they put the logs?

@snide
Copy link
Contributor Author

snide commented Mar 8, 2021

They live in the actions interface. You need to add a key to the repo to turn them on. I'd like to configure the messages first, then we should be safe to merge this, watch the log output as it runs (once every 8 hours) and then make some tweaks.

@kibanamachine
Copy link

Preview documentation changes for this PR: https://eui.elastic.co/pr_4625/

Copy link
Contributor

@cchaos cchaos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Interested in seeing the statistics in the end.

@snide
Copy link
Contributor Author

snide commented Mar 8, 2021

I have added a duplicate of this action on a shorter timer over in our gatsby-starter https://github.com/elastic/gatsby-eui-starter/actions/workflows/stale.yml

Once I see it successfully run and generating logs I'll merge this PR so we can see some dry-run stats for our meeting.

Copy link
Contributor

@thompsongl thompsongl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, let's try it out

@kibanamachine
Copy link

Preview documentation changes for this PR: https://eui.elastic.co/pr_4625/

@snide
Copy link
Contributor Author

snide commented Mar 8, 2021

Here's an example of the logs generated for elastic/gatsby-eui-starter

https://github.com/elastic/gatsby-eui-starter/runs/2061211848?check_suite_focus=true

I can see that it successfully ran in debug mode and took no action on the issues/prs. I'm gonna take that as reason this is safe to merge.

@snide snide added the platform label Mar 8, 2021
@kibanamachine
Copy link

Preview documentation changes for this PR: https://eui.elastic.co/pr_4625/

@snide snide merged commit 0f7f849 into elastic:master Mar 8, 2021
@snide snide deleted the actions/stale branch March 8, 2021 23:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog Use on PRs to skip changelog requirement (Don't delete - used for automation)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants