Code review là một trong những hoạt động có đòn bẩy cao nhất của một TL: nó bảo vệ chất lượng và truyền dạy. Mục tiêu là code tốt hơn và một nhóm mạnh hơn, chứ không phải gác cổng (gatekeeping). Các buổi review nên nhanh, tử tế và tập trung vào những gì quan trọng.
Các nguyên tắc
text
✓ Review CODE, không bao giờ review con người ("hàm này" chứ không phải "bạn")
✓ Phân biệt phải-sửa với nên-có (gắn nhãn rõ ràng cho các điểm nhỏ - nit)
✓ Đặt câu hỏi, đừng ra lệnh ("còn X thì sao?" mời gọi thảo luận)
✓ Khen ngợi việc làm tốt, không chỉ nêu vấn đề
✓ Nhanh — một PR bị chặn hai ngày sẽ giết chết đà tiến triển
✓ Approve khi nó đủ tốt, không cần hoàn hảo