Initial commit
This commit is contained in:
38
commands/pr-review.md
Normal file
38
commands/pr-review.md
Normal file
@@ -0,0 +1,38 @@
|
||||
---
|
||||
description: Reviews pull request changes to provide feedback, check for issues, and suggest improvements before merging into the main codebase.
|
||||
author: arkavo-org
|
||||
author-url: https://github.com/arkavo-org
|
||||
version: 1.0.0
|
||||
---
|
||||
|
||||
# Comprehensive PR Review Template
|
||||
|
||||
This is a comprehensive PR (Pull Request) review template with six distinct review tasks:
|
||||
|
||||
## 1. Product Manager Review
|
||||
- Focuses on business value, user experience, and strategic alignment
|
||||
|
||||
## 2. Developer Review
|
||||
- Evaluates code quality, performance, and adherence to best practices
|
||||
|
||||
## 3. Quality Engineer Review
|
||||
- Checks test coverage, potential bugs, and regression risks
|
||||
|
||||
## 4. Security Engineer Review
|
||||
- Assesses security vulnerabilities, data handling, and compliance
|
||||
|
||||
## 5. DevOps Review
|
||||
- Validates CI/CD pipeline, infrastructure, and monitoring considerations
|
||||
|
||||
## 6. UI/UX Designer Review
|
||||
- Ensures visual consistency, usability, and interaction flow
|
||||
|
||||
## Key Theme
|
||||
The document emphasizes an urgent, immediate approach to improvements, with repeated emphasis that "future" recommendations should be addressed right now, not deferred.
|
||||
|
||||
Each section follows a similar structure:
|
||||
- An objective
|
||||
- Specific areas to review
|
||||
- An action item requiring immediate implementation of any suggested improvements
|
||||
|
||||
The template is designed to provide a thorough, multi-perspective review of a software development pull request.
|
||||
Reference in New Issue
Block a user