เตรียมการเปลี่ยนแปลงให้พร้อมตรวจโค้ด

กระบวนการ

ทำตามลำดับนี้

1. ทวนผลผู้ใช้หรือระบบที่ตั้งใจ และเทียบส่วนต่างกับขอบเขตนั้น
2. ทบทวนทุกบรรทัดที่เปลี่ยนในบริบทไฟล์และระบบ เรื่องความเข้ากับการออกแบบ ความถูกต้อง ความปลอดภัย ความพร้อมกัน และความซับซ้อนที่เลี่ยงได้
3. ตรวจว่าการทดสอบล้มเหลวได้เมื่อพฤติกรรมพัง ครอบคลุมกรณีขอบที่เกี่ยวข้อง และสมกับระดับความเสี่ยงของการเปลี่ยน
4. รันการจัดรูปแบบ ลินต์ ตรวจชนิด การทดสอบ บิลด์ และการสร้างเอกสารที่ที่เก็บต้องมี
5. ตัดการเปลี่ยนบังเอิญ เสียงที่สร้าง ข้อความดีบัก โค้ดตาย และความลับ แยกการเปลี่ยนหากทบทวนรวมกันไม่ได้
6. เขียนสรุปคำขอรวมที่ครอบคลุมเหตุผล สิ่งที่เปลี่ยน หลักฐานตรวจ ภาพหน้าจอหรือโน้ตย้ายเมื่อเกี่ยวข้อง ความเสี่ยง และย้อนกลับ

เตรียมสิ่งนี้

ข้อมูลที่แมวจะขอ

• สาขาการเปลี่ยนหรือส่วนต่าง
• ประเด็นหรือเกณฑ์รับ
• คำสั่งร่วมสร้างและการทดสอบของที่เก็บ

เราเตอร์ + playbook เดียว

จัดเส้นทางก่อน เลือกอันเดียว

Cat Paw โหลด playbook เดียวต่องาน ไม่ผสมหลายอัน เริ่มที่เราเตอร์นี้ เส้นทางที่เหลือเป็นตัวเลือกเฉพาะ workflow

• เราเตอร์: engineering-pack
• skills/engineering/mattpocock/code-review/SKILL.md

เสร็จหมายถึง

จุดจบที่ตรวจได้

ส่วนต่างมีขอบเขตและอธิบายได้ การตรวจท้องถิ่นที่ต้องมีผ่าน อาร์ติแฟกต์ทบทวนครบ และไม่มีการค้นพบทบทวนตนเองความรุนแรงสูงที่ยังไม่ปิด

แหล่งที่มา

กระบวนการนี้มาจากไหน

1. google.github.io/eng-practices/review/reviewer/looking-for.html
2. google.github.io/eng-practices/review/developer/small-cls.html

แหล่งที่มาตั้งรูปแบบของ workflow โพสต์ Reddit และชุมชน ถ้ามี ให้ถือเป็นเรื่องเล่าจากผู้ปฏิบัติ ไม่ใช่ข้อเท็จจริงสากล