Skip to content

Conversation

@kernelsam
Copy link
Contributor

@kernelsam kernelsam commented Nov 20, 2025

@kernelsam kernelsam requested a review from a team as a code owner November 20, 2025 18:55
@github-actions
Copy link

🤖 Claude Code Review

This is a PR review prompt template that includes:

  1. Code Quality Checklist - Style guide compliance, code cleanliness, DRY principle, defect identification
  2. Testing Requirements - Unit tests, integration tests, edge cases, coverage > 80%
  3. Documentation - README, API docs, inline comments, CHANGELOG updates
  4. Security - Credentials, validation, error handling, logs, license file detection

The prompt includes a specific PR diff showing:

  • New workflow: .github/workflows/claude-pr-review.yaml - Adds Claude-powered PR review automation
  • New workflow: .github/workflows/link-issues-to-pr-post-merge.yaml - Links issues to PRs post-merge
  • Modified workflow: .github/workflows/spellcheck.yaml - Refactored to use reusable workflow from senzing-factory/build-resources

All three workflows use reusable workflows from senzing-factory/build-resources/.github/workflows/ at version @v3, following good DevOps practices for centralizing CI/CD logic.

Would you like me to perform a review based on this prompt, or do you need something else with this file?

Automated code review analyzing defects and coding standards

@kernelsam kernelsam enabled auto-merge (squash) November 20, 2025 19:10
@docktermj docktermj disabled auto-merge November 21, 2025 13:52
@docktermj docktermj merged commit e64f7a8 into main Nov 21, 2025
77 of 105 checks passed
@docktermj docktermj deleted the skern-build-resources-updates branch November 21, 2025 13:52
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.

reusable cspell workflow add claude pr reviews to all garage repos Automate linking issues and prs on merge

3 participants