Skip to content

Post heads-up notification when auto-creating monthly roundup issue#475

Merged
bph merged 2 commits into
trunkfrom
fix/roundup-heads-up-notification
May 18, 2026
Merged

Post heads-up notification when auto-creating monthly roundup issue#475
bph merged 2 commits into
trunkfrom
fix/roundup-heads-up-notification

Conversation

@bph
Copy link
Copy Markdown
Collaborator

@bph bph commented May 18, 2026

Summary

Fixes the missing heads-up notification on auto-created Monthly Roundup issues (e.g. #471).

Root cause

label-notifier.yml's initial-notification job listens for the labeled event on issues. However, GitHub Actions does not chain workflows triggered by GITHUB_TOKEN — so when create-roundup-issue.yml auto-creates the next month's issue with the Monthly Roundup label, the resulting labeled event is suppressed and label-notifier.yml never runs. The notification only fires when a human manually applies the label, which is why this wasn't caught earlier.

Fix

Post the heads-up comment directly inside create-roundup-issue.yml, right after the existing "update Google Doc / assign Author" comment. This bypasses the workflow-chaining limitation entirely and keeps label-notifier.yml working for the manual-label path.

The recipient list is hoisted into a workflow-level `env:` to make it easy to keep in sync with `label-notifier.yml`.

Test plan

  • After merge, close the current Monthly Roundup issue (What's new for developers? (June 2026) #471) once the May edition is published and verify the new issue receives both comments automatically.
  • Manually applying the `Monthly Roundup` label to a test issue still triggers `label-notifier.yml` (no regression).

bph added 2 commits May 18, 2026 11:22
The label-notifier workflow's initial-notification job listens for the
'labeled' event, but events fired by GITHUB_TOKEN inside another workflow
do not trigger downstream workflows. As a result, when create-roundup-issue
auto-creates the next month's issue with the Monthly Roundup label, the
heads-up @-mention notification never fires (see issue #471).

Post the heads-up comment directly from this workflow so the notification
runs reliably for both bot-created and manually-labeled issues.
Clarify why the initial-notification job appears redundant: the normal
monthly cycle is handled inline by create-roundup-issue.yml, and this
job is the fallback for manually-created or off-cycle roundup issues.
@bph bph merged commit 75eb8e7 into trunk May 18, 2026
@bph bph deleted the fix/roundup-heads-up-notification branch May 18, 2026 09:50
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.

1 participant