-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Check stale issues in ascending order #9489
Conversation
We have so many issues that only latest ones are checked for staleness due to rate limit
@@ -47,3 +47,4 @@ jobs: | |||
close-issue-message: > | |||
This issue has been closed because it has not received any activity in the last 14 days | |||
since being marked as 'stale' | |||
ascending: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It does make sense to close the older stale issues first. But I was wondering was there any default limit to number of issues to be processed? Why it is not closing all the issues that are older than 180 days and stale for 14 days.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The default value is 30 as per https://github.com/actions/stale#operations-per-run
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the workflow triggers daily at midnight, considering 30 issues a day. It should have closed by now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With the default descending order, it always checks the latest issues and only a small amount can be closed. Older issues are never checked.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, let's see if that makes a difference and old issues get closed
We have so many issues that only latest ones are checked for staleness due to rate limit
We have so many issues that only latest ones are checked for staleness due to rate limit
We have so many issues that only latest ones are checked for staleness due to rate limit
We have so many issues that only latest ones are checked for staleness due to rate limit.