Skip to content

ci: restore the labeled trigger on PR release labels - #325

Merged
flowcool merged 1 commit into
mainfrom
ci/restore-labeled
Oct 27, 2026
Merged

flowcool merged 1 commit into
mainfrom
ci/restore-labeled

Conversation

@flowcool

Copy link
Copy Markdown
Owner

Behavior

Follow-up to #324. That PR removed the labeled trigger from PR release labels on the assumption that the workflow re-triggered on its own label writes. Labels written with GITHUB_TOKEN never trigger workflows, so the removal saved nothing and dropped the Release impact re-check when breaking-change or compat is added by hand (finding from the Pullfrog review of the sibling boursedirect-sync PR). This restores the original trigger list.

Release impact

None. CI only.

Verification

AI assistance

Prepared by Claude Code (Opus 5.5).

🤖 Generated with Claude Code

https://claude.ai/code/session_01EQXdcx49Pi5uQErd6kDEKs

GITHUB_TOKEN label writes never trigger workflows, so removing labeled saved
nothing; it re-checks Release impact when breaking-change or compat is added
by hand.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01EQXdcx49Pi5uQErd6kDEKs
@coderabbitai

coderabbitai Bot commented Oct 27, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added the maintenance Dependencies, CI, chores label Oct 27, 2026

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ No new issues found.

Reviewed changes

Reviewed the single-commit change to the PR release labels workflow trigger.

  • Restore labeled trigger: .github/workflows/pr-release-labels.yml goes back to types: [opened, edited, reopened, labeled]. The change is safe: scripts/label_pr_release.py reads current labels from the event payload and only adds missing ones, and labels written with GITHUB_TOKEN don't re-trigger workflows, so there is no loop. The per-PR concurrency group still collapses overlapping runs.

Pullfrog  | View workflow run | Using claude-sonnet-5-5 | 𝕏

@flowcool
flowcool merged commit 781e594 into main Oct 27, 2026
16 checks passed
@flowcool
flowcool deleted the ci/restore-labeled branch October 27, 2026 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Dependencies, CI, chores

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant