Skip to content

merge queue: merged #614: chore(camNC) Add Claude PR review workflow and pre-push review rule - #761

Closed
mergify[bot] wants to merge 5 commits into
mainfrom
mergify/merge-queue/9e8e05543b
Closed

merge queue: merged #614: chore(camNC) Add Claude PR review workflow and pre-push review rule#761
mergify[bot] wants to merge 5 commits into
mainfrom
mergify/merge-queue/9e8e05543b

Conversation

@mergify

@mergify mergify Bot commented Nov 11, 2026 •
edited
Loading

Copy link
Copy Markdown

Note

This is a merge queue draft opened by Mergify. Its pull request has merged, and nobody needs to act on it.
Mergify opened it for batch inquisitive-painbeurre-pk4bw1 of the main queue, to test #614 before it merged into main.

🎉 What landed

Mergify merged its pull request into main, then closed this draft:

  • #614 chore(camNC) Add Claude PR review workflow and pre-push review rule (Schibum), as 91acf54

This draft's code was built in this order:

  1. main at 62734c6
  2. #614, the pull request of this batch

Bisected from a failed batch:

Metadata for CI tooling, read by mergify ci 
---
all_scopes: false
checking_base_sha: 62734c666d6fa72b1ca9329180d7b362c9a53939
previous_check_retries: []
previous_failed_batches:
  - batch_pr_head_sha: 1754503af0c0456443fc07017ddd10251b5d92c5
    batch_pr_number: 756
    checked_pull_requests:
      - 614
      - 633
      - 745
    draft_pr_head_sha: 1754503af0c0456443fc07017ddd10251b5d92c5
    draft_pr_number: 756
pull_requests:
  - number: 614
    scopes: []
scopes: []
...
Required conditions of queue rule main for merge
  • check-success = Behaviour specs (1/4)
  • check-success = Behaviour specs (2/4)
  • check-success = Behaviour specs (3/4)
  • check-success = Behaviour specs (4/4)
  • check-success = Run tests
  • check-success = Tests (app-browser-1)
  • check-success = Tests (app-browser-2)
  • check-success = Tests (app-browser-3)
  • github-review-approved [🛡 GitHub repository ruleset rule main]
  • any of [🛡 GitHub repository ruleset rule main]:
    • check-success = @github-actions/Run tests
    • check-neutral = @github-actions/Run tests
    • check-skipped = @github-actions/Run tests
Required conditions to stay in the queue

claude and others added 5 commits November 11, 2026 07:09
@claude
docs(camNC): Review the diff with a subagent before pushing to a PR
09e2776 
Every PR push runs CI, so catch bugs locally first.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QRhGQtkiBiWR8ZSACmv6PP
@claude
chore(camNC): Add Claude review workflow on the subscription token
430e9b6 
Reviews same-repo PRs when they open, reopen or leave draft, and on a
collaborator's "@claude review" comment. Not a required check.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QRhGQtkiBiWR8ZSACmv6PP
@claude
fix(camNC): Do not check out the PR head in the comment-triggered review
25b6a43 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QRhGQtkiBiWR8ZSACmv6PP
@claude
fix(camNC): Let @claude review re-runs review again
9719f82 
The code-review plugin stops when Claude has already commented, so a re-run after fixes did nothing. Also point it at AGENTS.md.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QRhGQtkiBiWR8ZSACmv6PP
@mergify @mergify-commit-signer
Merge of #614
fa89f8e
@mergify mergify Bot mentioned this pull request Nov 11, 2026
Merged
@mergify mergify Bot changed the title merge queue: checking #614 on main (62734c6) merge queue: merged #614: chore(camNC) Add Claude PR review workflow and pre-push review rule Nov 11, 2026
@mergify mergify Bot closed this Nov 11, 2026
@mergify
mergify Bot deleted the mergify/merge-queue/9e8e05543b branch November 11, 2026 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@claude