การตรวจสอบโค้ดเป็นหนึ่งในกิจกรรมที่มีอิทธิพลสูงสุดของ TL: มันป้องกันคุณภาพ และ สอนได้ เป้าหมายคือ โค้ดที่ดีกว่าและทีมที่แข็งแกร่ง ไม่ใช่การควบคุมเข้า การตรวจสอบควรเร็ว เป็นมิตร และเน้นสิ่งที่สำคัญ
หลักการ
text
✓ Review the CODE, never the person ("this function" not "you")
✓ Distinguish must-fix from nice-to-have (label nits explicitly)
✓ Ask questions, don't issue commands ("what about X?" invites discussion)
✓ Praise good work, not just problems
✓ Be fast — a PR blocked for two days kills momentum
✓ Approve when it's good enough, not perfect
ตัวอย่างที่เป็นรูปธรรม
แทนที่จะพูดว่า "นี่ผิด ใช้ map" ให้เขียน: "nit: map ที่นี่จะหลีกเลี่ยง nested loop ได้ คุ้มค่าไหม? ไม่มีการบล็อก" จุดเดียวกัน แต่มันสอน เปิดให้มีช่องว่างสำหรับการไม่เห็นด้วย และไม่ล็อก merge
