Skip to content

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

Open
sayakpaul wants to merge 6 commits into
mainfrom
improve-issue-reminder
Open

feat: improve issue reminder workflow and introduce conditional autoclose#14477
sayakpaul wants to merge 6 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
remind:
# Reminds external contributors to link an issue. PRs from maintainers, users
# with write/admin access, and collaborators are skipped by the script.
# Reminds external contributors to link an issue, sends still-unlinked PRs to a

@yiyixuxu yiyixuxu Aug 18, 2026

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.

i think:

(1) maybe it's easier (for us to keep track of the progress) if we do one message per PR? this way we can use a emoj ✅ to track it
(2) I think it's enough to send it once, if we do (1) and make sure all the previous message are read on our end
(3) maybe leave a bit longer from the slack message to auto-close?

just some thoughts, we can keep iterate after it's taken effect

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

  1. Agreed. It's better that way. I have addressed it in the latest commit.
  2. I think when a PR is not rescued and when we're autoclosing it, it's better to supplement a comment.
  3. Do you have an example? I think the current message is detailed enough?

@sayakpaul
sayakpaul requested a review from yiyixuxu August 18, 2026 08:51
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