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

Enable stale PR auto cleaning #681

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ciecierski
Copy link
Contributor

If a PR is open for more than 15 days without any activity, the bot will send a gentle reminder.

If nothing moves during the next 7 days, the bot will close the PR.

The same stale-pr workflow was introduced in ci-framework

If a PR is open for more than 15 days without any activity, the bot will
send a gentle reminder.

If nothing moves during the next 7 days, the bot will close the PR.

The same stale-pr workflow was introduced in ci-framework
@ciecierski
Copy link
Contributor Author

@jistr @holser There are already 30(including this one) open PRs. At least half of them with no activity in comments or new commits for some time already. What do you think about idea of closing stale PR requests? That way we can clearly see by looking at PR list what is actively being worked on, needs review etc.

@ciecierski
Copy link
Contributor Author

recheck

@jistr
Copy link
Contributor

jistr commented Oct 31, 2024

Perhaps this doesn't trigger the github check, i'm not really sure. I'd suggest waiting for after FR1 is released and then bypassing the branch protections to merge, and we'll see what happens. IMO it should be ok to merge but let's wait not to disrupt FR1, just in case...

@ciecierski ciecierski removed the request for review from holser November 6, 2024 14:17
@ciecierski
Copy link
Contributor Author

rdoproject.org/github-check is never going to run as .github/workflows is in irrelevant-files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants