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

🤖 Claude Code Review

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

Code Quality:

Testing:

  • Unit tests for new functions
  • Integration tests for endpoints
  • Edge case coverage
  • Test coverage > 80%

Documentation:

  • README updates
  • API documentation
  • Inline comments
  • CHANGELOG.md updates

Security:

  • No hardcoded credentials
  • Input validation
  • Error handling
  • No sensitive data in logs
  • Critical: Flag .lic files or strings starting with AQAAAD

Pull Request Diff:
The template includes a diff showing changes to three GitHub workflow files:

  1. .github/workflows/claude-pr-review.yaml (new) - Triggers Claude PR reviews
  2. .github/workflows/link-issues-to-pr-post-merge.yaml (new) - Links issues to PRs after merge
  3. .github/workflows/spellcheck.yaml (modified) - Refactored to use reusable workflow from senzing-factory/build-resources

Would you like me to perform a review based on this template, or help you with something specific related to this file?

Automated code review analyzing defects and coding standards

@kernelsam kernelsam enabled auto-merge (squash) November 20, 2025 19:38
@kernelsam kernelsam merged commit 24cfc6d into main Nov 20, 2025
27 checks passed
@kernelsam kernelsam deleted the skern-build-resources-updates branch November 20, 2025 20:07
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