Skip to content

merge queue: checking #704 on main (e1fd462) - #715

Closed
mergify[bot] wants to merge 4 commits into
mainfrom
mergify/merge-queue/504e9cdb8c
Closed

mergify[bot] wants to merge 4 commits into
mainfrom
mergify/merge-queue/504e9cdb8c

Conversation

@mergify

@mergify mergify Bot commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

🎉 This pull request has been checked successfully and will be merged soon. 🎉

#704 is queued for merge on branch main (e1fd462).

This pull request has been created by Mergify to check the mergeability of #704.
You don't need to do anything. Mergify will close this pull request automatically when it is complete.

Required conditions of queue rule default for merge:

  • check-success = gates (linux)
  • check-success = gates (windows)
  • check-success = red-first
  • github-review-approved [🛡 GitHub repository ruleset rule main-pr-required-checks]
  • label != do-not-merge
  • any of [🛡 GitHub repository ruleset rule main-pr-required-checks]:
    • check-success = @github-actions/gates (linux)
    • check-neutral = @github-actions/gates (linux)
    • check-skipped = @github-actions/gates (linux)
  • any of [🛡 GitHub repository ruleset rule main-pr-required-checks]:
    • check-success = @github-actions/gates (windows)
    • check-neutral = @github-actions/gates (windows)
    • check-skipped = @github-actions/gates (windows)
  • any of [🛡 GitHub repository ruleset rule main-pr-required-checks]:
    • check-success = @github-actions/red-first
    • check-neutral = @github-actions/red-first
    • check-skipped = @github-actions/red-first
  • any of [🛡 GitHub repository ruleset rule main-pr-required-checks]:
    • check-neutral = @mergify/Mergify Merge Protections
    • check-skipped = @mergify/Mergify Merge Protections
    • check-success = @mergify/Mergify Merge Protections

Required conditions to stay in the queue:

---
all_scopes: false
checking_base_sha: e1fd4627f49692ce94f34c4e2810ac757493d333
previous_check_retries: []
previous_failed_batches: []
pull_requests:
  - number: 704
    scopes: []
scopes: []
...

Cuarroc and others added 4 commits October 8, 2026 18:30
Test-First: scripts/ci/architecture-drift.test.mjs::rejects diagnosis commands in main.rs
Co-authored-by: Cursor <cursoragent@cursor.com>
This extracts get_log_path, get_panic_notice, get_reason_catalog,
export_diagnosis, reveal_log_path, get_stuck_after_minutes, and
set_stuck_after_minutes from main.rs into a new diagnosis_cmds.rs
module to shrink main.rs.

No-Test: covered by the previous commit's architectural rule
Co-authored-by: Cursor <cursoragent@cursor.com>
- Expand architecture-drift test to check the exact new location and generate_handler! export for all 7 diagnosis commands.
- Remove orphaned section headers in main.rs.

Test-First: scripts/ci/architecture-drift.test.mjs::contract: all 7 diagnosis commands are properly moved to diagnosis_cmds.rs and exported in main.rs
Co-authored-by: Cursor <cursoragent@cursor.com>
@mergify mergify Bot closed this Oct 8, 2026
@mergify
mergify Bot deleted the mergify/merge-queue/504e9cdb8c branch October 8, 2026 20:27
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