เวลามี Merge Request / Pull Request มีแนวทางการตรวจยังไง

พอดีเห็น Post ในกลุ่มสมาคมโปรแกรมเมอร์ไทย เลยคิดว่าเอา Comment ตัวเองในนั้นมองลอง Rewrite เขียน Blog ใหม่อีกทีน่าจะดีครับ ^__^ Prerequisite การ Review นอกจากช่วยทำให้ Code ดีขึ้นแล้ว ยังเป็นการแชร์ความรู้ด้วย แต่เราต้องมีพื้นฐาน หรือ Fundamential ที่ดีก่อนครับ แนวทางการตรวจ Merge Request / Pull Request มีหลายตัวที่ผมใช้นะ เอาไว้ตรวจตอน Review ส่วน AI ยังไม่เคยลองให้มันทำเต็มๆ นะ ถ้างง Copy ไปถามมัน เผื่อเป็นเทคนิคใหม่ๆ กรณีที่ MR / PR นั้นไฟล์มันมาแบบเยอะมาก ทำยังไง ? จากที่ผมเคยเจอหลัก 100 – 3000 ที่ผมเคยเจอมากสุดนะ ถ้าเริ่มต้องทำยังไง สุดท้ายแล้ว การที่ Merge มันมาเยอะๆ ขนาดนั้นต้องมองกลับไปที่การวางแผนแล้ว ว่าทำไมปล่อย ให้ Branch มาใหญ่ขนาดนั้นแล้ว Merge เข้าใจว่าจริงๆ มันควรทำให้เล็ก แต่มันมีหลายส่วนที่ Merge ต้องใหญ่ จากประสบการณ์ กลัว Code ชาวบ้านที่จะเข้ามาร่วมด้วย อันนี้ อาจจะต้องแก้ด้วยการทำ Test ให้ครอบคลุม และตอบสนองได้ไว อ๋อ แล้ว Code review ไม่ใช่การมานั่งจับผิดโค้ด แต่เป็นกระบวนการแบ่งปันองค์ความรู้ให้กันและกัน Key สำคัญของการทำ Code Review พยายามมองหาจุดที่สามารถ Improve Code หรือ ตรวจสอบ Impact ที่เกิดขึ้นได้ ทั้งในส่วน Technical หรือ ในส่วน Requirement เองเป็นต้น รวมถึงการแชร์ความรู้ทั้ง…








