เวลารีวิวโค้ด คุณมีเคล็ดลับส่วนตัวอะไรบ้างที่จะช่วยให้สังเกตเห็นสิ่งที่ 'ควรแก้แต่ไม่ได้แก้' ได้ดีขึ้น?

สำหรับ "การเปลี่ยนแปลง" ที่เกิดขึ้นใน commit หรือ PR หรือ MR นี้ การเขียนคอมเมนต์ว่าการเปลี่ยนแปลงนั้นโอเคหรือไม่ค่อนข้างทำได้ง่าย แต่การสังเกตเห็นส่วนที่ควรเปลี่ยนแต่ไม่ได้เปลี่ยนแล้วคอมเมนต์ไว้ แม้ว่าจะรีวิวโค้ดมาค่อนข้างนานแล้วก็ยังรู้สึกว่ายากอยู่เสมอ โดยเฉพาะถ้าไม่ใช่ codebase ที่คุ้นเคยมากสำหรับผม

ผมนึกวิธีออกมาบ้างเหมือนกัน แต่ถ้ามีเครื่องมือหรือหลักยึดมากกว่านี้ก็น่าจะดี เลยอยากรู้ว่าทุกคนทำกันอย่างไรบ้าง

• ตอน implement: ออกแบบให้ส่วนที่ต้องถูกเปลี่ยนร่วมกันในภายหลังสามารถถูกครอบคลุมด้วย test code ไปพร้อมกันได้
• ตอนเขียน PR: เพิ่มคำถามที่เหมาะสมให้ reviewee พิจารณาใน PR template (เช่น มี PR ก่อนหน้าที่มีจุดประสงค์คล้ายกับ PR นี้ไหม? เมื่อเทียบกับ PR นั้นแล้ว PR นี้มีส่วนไหนที่อาจตกหล่นไปหรือเปล่า?)
  • ทุกครั้งที่เกิดกรณี "ควรทำแต่ตกหล่น" ใน codebase นั้น ให้จดบันทึกไว้ และอัปเดตเนื้อหานั้นเป็นตัวอย่างใน PR template ทุกครั้ง
• ตอนรีวิว PR: เพิ่มไว้ใน review guide เพื่อให้ reviewer ลองค้นหาส่วนที่ตกหล่นในลักษณะคล้ายกับคำถามข้างต้น