Labsco
redis logo

pull-requests

✓ Official8,600

by redis · part of redis/redisinsight

When creating PRs with AI assistance, always add the "AI-Made" label.

🔥🔥✓ VerifiedFreeQuick setup
🔒 Repo-maintenance skill. It exists to help maintain redis/redisinsight itself — it's only useful if you contribute code to that project.

This is the playbook your agent receives when the skill activates — you don't need to read it to use the skill, but it's here to audit before installing.


name: pull-requests description: >- Create and review pull requests following project standards including title format, description template, and review checklist. Use when creating PRs, writing PR descriptions, or reviewing pull requests.

Pull Request Guidelines

Creating a PR

Labels

When creating PRs with AI assistance, always add the "AI-Made" label.

PR Title

Include issue number at the start:

RI-123 Add user profile editing
#456 Fix memory leak in connection pool

PR Description Template

# What

Describe what was changed.

# Testing

Describe how to test the changes.

---

Closes #RI-123

PR Description Guidelines:

  • Keep it concise - Avoid verbose descriptions
  • Focus on high-level changes - Don't list every code change in the #What section
  • Brief and to the point - The diff shows the details; describe the "why" and "what" at a high level
  • Technical decisions - Only mention significant architectural or design decisions if relevant

Review Process

As PR Author

  • Respond to all comments - Address every piece of feedback
  • Don't take feedback personally - Reviews improve code quality
  • Update code based on feedback - Make requested changes
  • Mark conversations as resolved - After addressing feedback
  • Keep PR up to date - Rebase on main regularly

As PR Reviewer

  • Be constructive and respectful - Focus on improvement
  • Focus on logic, not style - Linter handles formatting
  • Check for:
    • Logic errors and edge cases
    • Performance issues
    • Security concerns
    • Test coverage
    • Missing documentation
    • Architectural concerns

Review Checklist

  • Code follows project patterns
  • Tests are comprehensive
  • No console.log or debug code
  • TypeScript types are proper
  • Error handling is adequate
  • Documentation is updated
  • No security vulnerabilities
  • Performance is acceptable