-
Notifications
You must be signed in to change notification settings - Fork 249
Fix stale workflow to never close issues and ignore dependabot PRs #3717
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
base: master
Are you sure you want to change the base?
Conversation
…sing Co-authored-by: rbtr <[email protected]>
.github/workflows/stale.yaml
Outdated
@@ -16,14 +16,15 @@ jobs: | |||
id: stale | |||
with: | |||
ascending: true | |||
close-issue-message: 'Issue closed due to inactivity.' | |||
close-pr-message: 'Pull request closed due to inactivity.' | |||
days-before-close: 7 |
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.
@copilot does this close issues after 7 days still?
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.
Pull Request Overview
This PR updates the stale workflow so that issues are never auto-closed, Dependabot PRs are ignored, and the exempt-stale
label applies to both issues and PRs.
- Removed custom issue closing message and tweaked stale-issue wording to keep issues open indefinitely
- Added
exempt-pr-labels
andexempt-pr-authors
to skip Dependabot and other specified PRs - Retained PR closing cycle and messages for non-exempt PRs
Comments suppressed due to low confidence (1)
.github/workflows/stale.yaml:20
- Removing only the
close-issue-message
does not disable issue closing—days-before-close: 7
still applies and issues will close with the default message. To prevent closures entirely, either remove or setdays-before-close
appropriately or use the action's explicit flag to disable issue closing.
days-before-close: 7
|
… closing Co-authored-by: rbtr <[email protected]>
You're absolutely correct. Fixed in 07f7bb9 by properly using Warning Firewall rules blocked me from connecting to one or more addressesI tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
This pull request is stale because it has been open for 2 weeks with no activity. Remove stale label or comment or this will be closed in 7 days |
Updates the stale workflow to address three critical issues:
Issues Fixed
1. Issues Are Never Closed Due to Staleness
close-issue-message
parameter to prevent automatic issue closingstale-issue-message
to clarify that issues will not be auto-closed2. Dependabot PRs Are Properly Ignored
exempt-pr-authors: 'dependabot[bot]'
to exempt all dependabot PRs by authordependencies
andci
labels toexempt-pr-labels
to catch dependabot PRs by label3. exempt-stale Label Respected for Both Issues and PRs
exempt-pr-labels: 'exempt-stale,dependencies,ci'
to respect exempt-stale label for PRsexempt-issue-labels: 'exempt-stale'
for issuesBehavior Changes
Before:
After:
Testing
Fixes #3716.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.