Skip to content

ci: read Claude review model from repo variable#280

Open
fan-zhang-sv wants to merge 1 commit intomasterfrom
ci/claude-review-model-var
Open

ci: read Claude review model from repo variable#280
fan-zhang-sv wants to merge 1 commit intomasterfrom
ci/claude-review-model-var

Conversation

@fan-zhang-sv
Copy link
Copy Markdown
Collaborator

@fan-zhang-sv fan-zhang-sv commented Apr 17, 2026

Summary

  • Swap the hardcoded --model claude-opus-4-6-default in .github/workflows/claude-review.yml for ${{ vars.CLAUDE_MODEL }} so the model used by the Claude Code Review workflow can be managed via a repo Actions variable.
  • Matches the existing pattern already used for LLM_GATEWAY_HOSTNAME in the same workflow.

Test plan

  • Set CLAUDE_MODEL repo variable.
  • Open/sync a PR and confirm the Claude Code Review job passes and uses the configured model.

Made with Cursor

Replace the hardcoded `claude-opus-4-6-default` model with
`${{ vars.CLAUDE_MODEL }}` so the model can be updated via the repo
Actions variable without changing the workflow. Matches the existing
convention used for `LLM_GATEWAY_HOSTNAME`.

Made-with: Cursor
@cb-heimdall
Copy link
Copy Markdown
Collaborator

cb-heimdall commented Apr 17, 2026

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@dimabro848
Copy link
Copy Markdown

great

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.

3 participants