38 lines
1.4 KiB
Markdown
38 lines
1.4 KiB
Markdown
---
|
|
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. |