Remove unused ANTHROPIC_BASE_URL secret input#5
Conversation
- 🔥 The `workflow_call` declared it as a secret, but the step uses `vars.ANTHROPIC_BASE_URL` (a repo variable) instead Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e33d7c3beb
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
- 📝 Move `ANTHROPIC_BASE_URL` from Secrets table to a new Variables section - 🔥 Remove commented-out secret line from usage example Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
This PR simplifies the reusable Code Review GitHub Actions workflow by removing the unused ANTHROPIC_BASE_URL secret from workflow_call inputs, aligning the interface with current usage (vars.ANTHROPIC_BASE_URL).
Changes:
- Remove
ANTHROPIC_BASE_URLfromon.workflow_call.secretsin the reusable workflow.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
ANTHROPIC_BASE_URLsecret fromworkflow_callinputs — the step usesvars.ANTHROPIC_BASE_URL(a repo variable) instead, so the secret was never referencedTest plan
pull_requestandworkflow_callANTHROPIC_BASE_URLis still passed viavars.*at runtime🤖 Generated with Claude Code