Skip to content

ci: add Claude Code PR review and dependency-bump workflows#64

Merged
g-carre merged 1 commit into
mainfrom
improvement/setup-claude
Jun 15, 2026
Merged

ci: add Claude Code PR review and dependency-bump workflows#64
g-carre merged 1 commit into
mainfrom
improvement/setup-claude

Conversation

@g-carre

@g-carre g-carre commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

What

Sets up automated Claude Code review for the repo:

  • CLAUDE.md — repo context (Go RAID library, hexagonal architecture, vendor-CLI output parsing, pkg/errors + sentinel-error conventions, testify/mockery testing) plus Go coding standards.
  • .claude/skills/review-pr/SKILL.md — repo-specific /review-pr skill with a review-criteria table tailored to this codebase (error wrapping, hexagonal boundaries, port-interface compliance, CLI output parsing, command-injection safety, byte/enum conversions, mockery regeneration).
  • .github/workflows/review.yml — two jobs calling the reusable workflows from scality/workflows@v2.8.3:
    • review — code review for human PRs (pull_request), skipped for bot authors.
    • review-dependency-bump — dependency review for Dependabot and Scality Renovate (scality-renovate[bot]) PRs, via pull_request_target.

Required org config

For the workflows to run green, the repo needs access to:

  • Secrets: GCP_WORKLOAD_IDENTITY_PROVIDER, GCP_SERVICE_ACCOUNT, ANTHROPIC_VERTEX_PROJECT_ID, CLOUD_ML_REGION, ACTIONS_APP_PRIVATE_KEY
  • Variable: ACTIONS_APP_ID

Notes

  • review-dependency-bump uses pull_request_target (runs with base-repo secrets); the if: guards restrict it to dependency-bot PRs, and the called workflow is pinned to @v2.8.3.

🤖 Generated with Claude Code

Set up automated code review for the repo:

- CLAUDE.md: repo context (Go RAID library, hexagonal architecture,
  vendor-CLI parsing, error/test conventions) plus Go coding standards.
- .claude/skills/review-pr/SKILL.md: repo-specific /review-pr skill with
  review criteria tailored to this codebase.
- .github/workflows/review.yml: code review for human PRs, and
  dependency-bump review for Dependabot and Scality Renovate
  (scality-renovate[bot]) PRs via pull_request_target.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@g-carre g-carre requested a review from a team as a code owner June 15, 2026 13:32
@g-carre g-carre merged commit 5598c44 into main Jun 15, 2026
5 checks passed
@g-carre g-carre deleted the improvement/setup-claude branch June 15, 2026 13:58
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