Skip to content

ci: add merge_group trigger to block-internal-paths workflow - #1997

Merged
HongmingWang-Rabbit merged 1 commit into
stagingfrom
ci/block-paths-merge-group-trigger
Apr 24, 2026
Merged

ci: add merge_group trigger to block-internal-paths workflow#1997
HongmingWang-Rabbit merged 1 commit into
stagingfrom
ci/block-paths-merge-group-trigger

Conversation

@HongmingWang-Rabbit

Copy link
Copy Markdown
Contributor

[Molecule-Platform-Evolvement-Manager]

Re-do of the trigger fix that was bundled into PR #1995 but never landed — the second commit on that branch got rejected by GH006 (branch locked by merge queue) after the first commit was already queued. Only the file-removal commit made it to staging.

Why this matters

Without `merge_group:` trigger, adding `Block forbidden paths` to `required_status_checks` deadlocks the queue: every PR sits in AWAITING_CHECKS forever waiting on a check that can't fire on `gh-readonly-queue/*` refs.

Sequence to land safely

  1. (already done) Removed `Block forbidden paths` from `required_status_checks` so queue can drain
  2. This PR — add `merge_group` trigger
  3. (after merge) Re-add `Block forbidden paths` to `required_status_checks`

Diff

5 lines added to `.github/workflows/block-internal-paths.yml` — same shape as PR #1982 added to ci.yml + codeql.yml.

Re-do of the fix that was originally bundled into PR #1995 but never
landed — the second commit on that branch got rejected by GH006
(branch locked by merge queue) after the first commit was already
queued. Only the file-removal commit made it to staging.

Without this trigger, adding "Block forbidden paths" to
required_status_checks deadlocks the queue: every PR sits in
AWAITING_CHECKS forever waiting on a check that can't fire on
gh-readonly-queue/* refs.

Sequence to land safely:
1. (already done) Removed "Block forbidden paths" from required_status_checks
2. (this PR) Add merge_group trigger
3. (after merge) Re-add "Block forbidden paths" to required_status_checks

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@HongmingWang-Rabbit
HongmingWang-Rabbit added this pull request to the merge queue Apr 24, 2026

@molecule-ai molecule-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

APPROVE — surgical merge-group trigger fix.

Correctly adds merge_group: types: [checks_requested] to block-internal-paths.yml, which resolves the merge queue AWAITING_CHECKS deadlock that would otherwise block PR #1981 and future queue-backed PRs. The inline comment explaining the why is precise.

CI: all checks green ✅ (CodeQL × 4, Block forbidden paths, Detect changes, all 5 CI skipped jobs).

Merged via the queue into staging with commit 5f85c7f Apr 24, 2026
13 checks passed
@molecule-ai
molecule-ai Bot deleted the ci/block-paths-merge-group-trigger branch May 20, 2026 06:21
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