Skip to content

feat: improve issue reminder workflow and introduce conditional autoclose - #14477

Merged
sayakpaul merged 11 commits into
mainfrom
improve-issue-reminder
Aug 22, 2026
Merged

sayakpaul merged 11 commits into
mainfrom
improve-issue-reminder

Conversation

@sayakpaul

Copy link
Copy Markdown
Member

As discussed internally.

@github-actions github-actions Bot added CI size/M PR with diff < 200 LOC labels Aug 14, 2026
@sayakpaul
sayakpaul requested a review from yiyixuxu August 14, 2026 13:57
Comment thread .github/workflows/pr_link_issue_reminder.yml Outdated
@sayakpaul
sayakpaul requested a review from yiyixuxu August 18, 2026 08:51

@yiyixuxu yiyixuxu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

thanks, i left some feedbacks
feel free to merge once you addressed them, or not (we can experiment and see how it goes)

Comment thread utils/remind_link_issue.py Outdated
Comment thread utils/remind_link_issue.py Outdated
Comment thread utils/remind_link_issue.py Outdated
sayakpaul and others added 4 commits August 22, 2026 08:22
The Slack rescue branch fired on every daily run between SLACK_RESCUE_DAYS and
AUTOCLOSE_DAYS, so a PR the team decided not to rescue came back three times.
Append a hidden marker to the warning comment once a PR has been reported, and
skip PRs that already carry it.

Also move the Slack ping from 7 to 5 days after the warning, leaving 5 days
(instead of 3) between the message and the auto-close, so a Friday message
still gets a reply.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@sayakpaul
sayakpaul merged commit 0a0c57d into main Aug 22, 2026
16 checks passed
@sayakpaul
sayakpaul deleted the improve-issue-reminder branch August 22, 2026 03:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI size/M PR with diff < 200 LOC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants