Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 0 additions & 10 deletions .github/workflows/claude-review-hosted.yml
Original file line number Diff line number Diff line change
Expand Up @@ -75,16 +75,6 @@ jobs:
pull-requests: write # post review + tracking comment
id-token: write # OIDC, mints the Claude GitHub App token (required
# even with an OAuth/API-key credential)
# Repo-wide review queue: a SEPARATE block from the workflow-level
# cancel group above, because `queue: max` cannot be combined with
# `cancel-in-progress: true`, the value that block sets
# (https://docs.github.com/en/actions/reference/workflows-and-actions/workflow-syntax#jobsjob_idconcurrency),
# and because this group is job-scoped and repo-wide, not per-PR.
# Serializes reviews repo-wide so parallel PRs queue for the shared
# Claude seat instead of contending for it.
concurrency:
group: claude-review-${{ github.repository }}
queue: max
uses: melodic-software/ci-workflows/.github/workflows/claude-review.yml@35880dcbb2f174aac90159e276dc7eddf1bc20b9 # v0.30.1
with:
runner: ubuntu-24.04
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/claude-security-review-hosted.yml
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ permissions:
# queue group: a full queue CANCELS new arrivals, which is the same wedge; a
# queue is added only if overflow smoke-testing proves otherwise. These are
# per-LANE values, deliberately different from the code-review caller's (which
# cancels superseded runs and queues repo-wide). Do not normalize the two.
# cancels superseded runs). Do not normalize the two.

jobs:
security-review:
Expand Down
Loading