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
Một ví dụ cụ thể
Thay vì "Cái này sai, dùng map đi," hãy viết: "nit: một map ở đây sẽ tránh được vòng lặp lồng nhau, có đáng không? Không chặn merge nhé." Cùng một ý, nhưng nó truyền dạy, để chỗ cho người ta phản biện, và không chặn việc merge.