FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add: Claude automated PR review by RobotEagle · Pull Request #2 · EENCloud/syslinux · GitHub

Add: Claude automated PR review - #2

Open
RobotEagle wants to merge 1 commit into
mainfrom
chore/enroll-claude-review
Open

Add: Claude automated PR review#2
RobotEagle wants to merge 1 commit into
mainfrom
chore/enroll-claude-review

Conversation

Copy link
Copy Markdown

Enrolls this repo in automated Claude PR reviews via the claude_auto_review reusable workflow in workflow-library.

Every PR will be reviewed by RobotEagle (Claude) automatically on open, when marked ready for review, or when a review is requested from RobotEagle. To request a re-review after new commits, tag @claude in a PR comment or re-request a review from RobotEagle.

Before merging — Vault setup required

The workflow fetches the team's Anthropic API key from Vault. The key name is derived from the ownerId in compass.yml:

# Run this against the repo's compass.yml to find the Vault key name:

OWNER_ID=$(grep -m1 '^ownerId:' compass.yml | sed 's/^ownerId: *//' | tr -d '"' | xargs)

UUID=$(echo "$OWNER_ID" | sed 's/.*\///')

echo "team_$(echo "$UUID" | tr '-' '_')"

Ask a Vault admin to add the Anthropic API key at secret/github/actions/claude under that key name. If your team's key is already registered (another repo in the same team is already enrolled), no action is needed.

Remove any existing Claude review workflows

If this repo already has a workflow that triggers Claude PR reviews (e.g. a claude.yml calling claude_auto_review.yml), delete it when merging this PR. Keeping both will cause every PR to receive two reviews.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL