chore: switch merge-schedule to reminder workflow#51
Merged
ajbozarth merged 1 commit intoMay 21, 2026
Merged
Conversation
The scheduled-merge workflow can no longer auto-enqueue PRs once merge queue is enabled — actions taken by GITHUB_TOKEN don't fire merge_group events, so CI never runs on the queued ref and the queue evicts the PR. Bump the workflow pin to the new reminder-only version, drop the contents:write permission, and update CONTRIBUTING/AGENTS to describe the new behavior. Also recommend setting /remind YYYY-MM-DD at PR open matching the front matter date, and document that the date should be future-dated to reflect when the post will go live. Assisted-by: Claude Code Signed-off-by: Alex Bozarth <ajbozart@us.ibm.com>
ajbozarth
force-pushed
the
chore/merge-schedule-reminder
branch
from
687a4f6 to
e61b256
Compare
psschwei
approved these changes
May 21, 2026
Member
|
Just want to confirm, this wasn't a required check (and thus doesn't need to be updated in the branch protections) right? |
Contributor
Author
No, its just a cron, and even if it wasn't this removed the perms it did use to "write" now it just posts comments |
Merged
via the queue into
generative-computing:main
with commit May 21, 2026
307b88e
6 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Switch the scheduled-merge workflow from auto-enqueueing PRs to posting a reminder comment, and update author docs to match.
The old behavior fails silently with merge queue enabled: the workflow successfully calls
gh pr merge --auto, but because the actor isgithub-actions[bot], the resultingmerge_groupevent is suppressed by GitHub's "GITHUB_TOKEN can't trigger workflows" rule. CI never runs on the queued ref and the queue evicts the PR after ~1 hour. Confirmed against PR #36 — enqueued by the workflow at 15:12, evicted at 16:15 with zero CI runs in between, then manually re-enqueued by a human and merged in 13s oncemerge_groupactually fired.The reminder-only model has the bot identify due PRs and ping the author/assignees; a human does the enqueue gesture so
merge_groupfires correctly.Changes
.github/workflows/merge-schedule.ymlto generative-computing/.github@d04be31 (the new reminder-only reusable workflow).contents: writepermission from the caller (no longer merging anything).Scheduled Merge→Scheduled Merge Reminder) and job (merge-scheduled-prs→remind-scheduled-prs) to match the upstream rename.CONTRIBUTING.mdandAGENTS.mdto:/remind YYYY-MM-DD(the new keyword; legacy/schedulestill works upstream) at PR open, matching the front matterdate.dateshould be future-dated to reflect when the post goes live, not when it was drafted.Test plan
workflow_dispatchtrigger is wired.workflow_dispatchwith a test PR carrying a past-dated/remind, verify a reminder comment appears mentioning the author./remindis skipped on the same run.workflow_dispatchrun on the same PR posts a fresh reminder (daily nudges, no idempotency suppression).