Review Pull Request

gh-review-pr

Comprehensively review a pull request including code changes, CI status, and adherence to project standards. Use when asked to review a PR.

majiayu000/claude-skill-registry·0installs663starsSecurity

SKILL.md

Full skill instructions

Review Pull Request

Evaluate PR quality and provide structured feedback.

When to Use

Reviewing a PR before merge
Evaluating code quality and standards
Checking CI status and test coverage
Providing feedback to contributors

Quick Reference

# View PR details
gh pr view <pr>

# Check diff
gh pr diff <pr> | less

# View CI status
gh pr checks <pr>

# See review comments
gh api repos/OWNER/REPO/pulls/PR/comments

Review Checklist

Code follows project standards (CLAUDE.md)
All CI checks passing
Tests are present and passing
PR is linked to issue
Commit messages follow conventional commits
No security vulnerabilities
Documentation updated if needed
No unintended files included

Workflow

Get PR info: gh pr view <pr>
Review diff: gh pr diff <pr>
Check CI: gh pr checks <pr>
Analyze code: Focus on quality, standards, tests
Assess security: Look for vulnerabilities
Verify docs: Check if documentation updated
Provide feedback: Comment with findings

Review Focus Areas

Code Quality:

Clean, readable, maintainable
Follows CLAUDE.md guidelines
Proper error handling
No code duplication

Testing:

Tests present for new code
Tests passing
Adequate coverage
Edge cases covered

Documentation:

API documentation updated
Complex logic explained
README updated if needed
Examples provided if applicable

Standards Compliance:

Mojo syntax correct (for Mojo code)
No warnings or unused variables
Proper formatting
Conventional commit messages

Output Format

Provide review with sections:

Summary - Overall assessment
Strengths - What's done well
Issues - Problems that must be fixed
Suggestions - Optional improvements
Verdict - Approve / Request Changes / Comment

Error Handling

Problem	Solution
PR not found	Verify PR number
Auth failure	Check `gh auth status`
CI pending	Note review based on current state

References

See CLAUDE.md for project standards
See CLAUDE.md for Mojo syntax requirements
See CLAUDE.md for zero-warnings policy

Install skill

Copy and run in your terminal

npx -y skills add https://github.com/majiayu000/claude-skill-registry --skill gh-review-pr

View on GitHub

skills/data/gh-review-pr/SKILL.md

More Security skills

View all

Security

supabase

Handles the full Supabase workflow from schema changes to deployment, with built-in security guardrails that catch common traps like RLS...

2.7K 308.9K

View

Security

neon-postgres

Comprehensive guides and best practices for Neon Serverless Postgres, covering setup, connection methods, authentication, and platform APIs.

98 216.2K

View

Security

firebase-auth-basics

Guide for setting up and using Firebase Authentication. Use this skill when the user's app requires user sign-in, user management, or secure data access using auth rules.

462 163.8K

View

Security

firebase-security-rules-auditor

A skill to evaluate how secure Firestore security rules are. Use this when Firestore security rules are updated to ensure that the generated rules are extremely secure and robust.

462 127.7K

View

Security

firebase-ai-logic-basics

Official skill for integrating Firebase AI Logic (Gemini API) into web applications. Covers setup, multimodal inference, structured output, and security.

462 125.1K

View

Security

better-auth-best-practices

Complete Better Auth server and client setup with database adapters, session management, plugins, and security configuration.

222 118.2K

View

Security

Deploy and manage projects on Vercel using token-based authentication. Use when working with Vercel CLI using access tokens rather than interactive login — e.g. "deploy to vercel", "set up vercel", "add environment variables to vercel".

31.9K 116.1K

View

Security

cloudflare

Complete Cloudflare platform integration with decision trees for compute, storage, AI, networking, security, and infrastructure-as-code.

3K 110.7K

View

Security

azure-compliance

Run Azure compliance and security audits with azqr plus Key Vault expiration checks. Covers best-practice assessment, resource review, policy/compliance validation, and security posture checks. WHEN: compliance scan, security audit, BEFORE running azqr (compliance cli tool), Azure best practices, Key Vault expiration check, expired certificates, expiring secrets, orphaned resources, compliance assessment.

253 103K

View