Skip to content

chore(ci): call the shared claude-review workflow - #16

Merged
jfrench9 merged 1 commit into
mainfrom
chore/shared-claude-review
Sep 2, 2026
Merged

jfrench9 merged 1 commit into
mainfrom
chore/shared-claude-review

Conversation

@jfrench9

@jfrench9 jfrench9 commented Sep 2, 2026

Copy link
Copy Markdown
Member

Reduces claude.yml to its triggers plus the permissions grant ( 50 → 28 lines). The author gate, runner selection, and the claude-code-action / checkout pins now come from RoboFinSystems/robosystems/.github/workflows/claude-review.yml.

Also picks up the SHA-pinned checkout — this repo was on the mutable actions/checkout@v7 tag.

No behaviour change: the gate and permissions are identical, and select-runner returns ["ubuntu-latest"] without an API call while RUNNER_LABELS is github-hosted.

Verified end to end in robosystems (run 33592717239: check-runner-availabilityclaude, both green) and piloted in robosystems-python-client#206.

Reduces claude.yml to its triggers and permissions grant; the author
gate, runner selection and the claude-code-action and checkout pins now
come from RoboFinSystems/robosystems/.github/workflows/claude-review.yml,
where they are maintained once rather than in each repo.

Also picks up the SHA-pinned checkout from the shared workflow - this
repo was on the mutable actions/checkout@v7 tag.

No behaviour change: the gate and permissions are identical, and
select-runner returns ["ubuntu-latest"] without an API call while
RUNNER_LABELS is github-hosted.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0113UQ7evQKHNf1L1tFiZzLD
@jfrench9
jfrench9 merged commit 291cf89 into main Sep 2, 2026
4 checks passed
@jfrench9
jfrench9 deleted the chore/shared-claude-review branch September 4, 2026 03:39
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