(BSR) Add claude code review for runner-manager#713
Merged
matthiasL-scality merged 1 commit intomainfrom Apr 13, 2026
Merged
Conversation
charlesprost
approved these changes
Apr 10, 2026
ef009c5 to
50ccbc5
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #713 +/- ##
=======================================
Coverage 85.79% 85.79%
=======================================
Files 36 36
Lines 1809 1809
=======================================
Hits 1552 1552
Misses 257 257
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
11230f4 to
9970ea3
Compare
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
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces automated code review capabilities for the
runner-managerproject using Claude and GitHub Actions. It adds a reusable workflow to trigger reviews on pull requests, configures the Claude plugin environment, and documents the review process and project structure. The most important changes are grouped below:Automated code review workflow:
.github/workflows/review.yml) that triggers on pull requests and delegates the review job to a shared workflow, passing required cloud and Anthropic credentials as secrets.Claude plugin and skill configuration:
scality-skills@scality-agent-hubplugin in the Claude settings (.claude/settings.json).review-prskill (.claude/skills/review-pr/SKILL.md) that defines how Claude reviews pull requests, including criteria for code quality, error handling, type hints, security, and more. The skill supports both CI (automated) and local (manual) review modes, and details the expected review output format.Project documentation:
CLAUDE.mdfile describing the purpose, architecture, and directory structure of therunner-managerservice, as well as its technology stack and integration points.