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:57
@github-actions
Copy link

🤖 Claude Code Review

This file contains a PR code review prompt template with a checklist covering:

  1. Code Quality - Style guide compliance, no commented code, meaningful names, DRY principle, and defect identification
  2. Testing - Unit/integration tests, edge cases, and coverage requirements
  3. Documentation - README, API docs, inline comments, and CHANGELOG updates
  4. Security - No hardcoded credentials, input validation, error handling, no sensitive data in logs, and flagging license files

The bottom section shows a Pull Request Diff for changes that:

  • Add a new Claude PR review workflow (.github/workflows/claude-pr-review.yaml)
  • Add a workflow to link issues to PRs post-merge (.github/workflows/link-issues-to-pr-post-merge.yaml)
  • Refactor the spellcheck workflow to use a reusable workflow from senzing-factory/build-resources

These changes appear to be adding automated code review and issue linking functionality, plus consolidating the spellcheck workflow to use a centralized version.

Would you like me to perform a review of this PR against the checklist, or are you looking for something specific?

Automated code review analyzing defects and coding standards

@kernelsam kernelsam merged commit 880feda into main Nov 20, 2025
21 checks passed
@kernelsam kernelsam deleted the skern-build-resources-updates branch November 20, 2025 19:50
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

2 participants