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

🤖 Claude Code Review

This file contains a PR code review prompt template with a checklist for reviewing pull requests. It includes:

  1. Code Quality checklist - style guide compliance, no commented-out code, meaningful variable names, DRY principle, and defect identification

  2. Testing checklist - unit tests, integration tests, edge cases, and test coverage requirements (>80%)

  3. Documentation checklist - README updates, API docs, inline comments, and CHANGELOG.md updates

  4. Security checklist - no hardcoded credentials, input validation, proper error handling, no sensitive data in logs, and a specific flag for license files (.lic) containing strings starting with AQAAAD

  5. Pull Request Diff section - Contains an example diff showing:

    • Addition of .github/workflows/claude-pr-review.yaml - a new workflow for Claude PR reviews
    • Addition of .github/workflows/link-issues-to-pr-post-merge.yaml - a workflow to link issues after PR merge
    • Modification of .github/workflows/spellcheck.yaml - refactored to use a reusable workflow from senzing-factory/build-resources

The template asks for ✅/❌ assessment of each checklist item with explanations for any issues found. Would you like me to help you use this template to review a PR, or modify the template itself?

Automated code review analyzing defects and coding standards

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