Skip to content

general-pr-review: declare ANTHROPIC_API_KEY for workflow_call - #120

Merged
jugonzalez12 merged 1 commit into
mainfrom
jugonzalez/declare-pr-review-secret
Sep 16, 2026
Merged

jugonzalez12 merged 1 commit into
mainfrom
jugonzalez/declare-pr-review-secret

Conversation

@jugonzalez12

Copy link
Copy Markdown
Contributor

The workflow_call block declared no secrets, so a caller could only reach ANTHROPIC_API_KEY through secrets: inherit, which forwards every secret the calling repository can see. Declaring it lets callers pass that one secret instead.

No behaviour change for existing consumers. The managed-repository ruleset runs this workflow against each repo through its own pull_request trigger rather than workflow_call, so the declaration is inert there and the secret keeps resolving from the org secret. A code search across ConductorOne and ductone finds no workflow_call references to this file today.

The workflow_call block declared no secrets, so a caller could only reach
ANTHROPIC_API_KEY through secrets: inherit, which forwards every secret the
calling repository can see. Declaring it lets callers pass that one secret
instead.

No behaviour change for existing consumers. The managed-repository ruleset
runs this workflow against each repo through its own pull_request trigger
rather than workflow_call, so the declaration is inert there and the secret
keeps resolving from the org secret. A code search across ConductorOne and
ductone finds no workflow_call references to this file today.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@jugonzalez12
jugonzalez12 merged commit dcc7efb into main Sep 16, 2026
2 checks passed
@jugonzalez12
jugonzalez12 deleted the jugonzalez/declare-pr-review-secret branch September 16, 2026 18:38
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.

2 participants