| name | code-review-review-checklist |
| description | null |
Review Checklist
Category: code-review · Status: 🟢 Active
When to use
Khi review PR chung: đảm bảo đúng, rõ ràng, an toàn, có test.
Steps
- Đúng: code làm đúng yêu cầu, phủ edge case, xử lý lỗi.
- Rõ: tên biến/hàm dễ hiểu, không lặp, complexity hợp lý, không code chết.
- An toàn: validate input, không lộ secret, xử lý quyền (xem security-review).
- Test: có test cho thay đổi, phủ nhánh chính, chạy pass.
- Phạm vi: PR nhỏ, đúng mục tiêu, không lẫn refactor không liên quan.
Template
## Review
- [ ] Đúng yêu cầu + edge case + lỗi
- [ ] Tên/cấu trúc rõ, không trùng lặp
- [ ] Input validate, không lộ secret
- [ ] Có test, pass CI
- [ ] PR nhỏ, đúng scope
Nhận xét: <blocking> / <nit> / <gợi ý>
Example
Good: comment phân loại blocking vs nit, chỉ rõ vị trí + lý do + đề xuất.
Avoid: review "LGTM" không đọc, bắt bẻ style chủ quan, trộn nhiều mối quan tâm.
Checklist