ตรวจสอบการเปลี่ยนแปลง

เอกสารนี้จะอธิบายวิธีตรวจสอบการเปลี่ยนแปลง โปรดคำนึงถึงเกณฑ์ต่อไปนี้สำหรับการเปลี่ยนแปลงที่ถูกต้อง

  • การเปลี่ยนแปลงนี้เป็นไปตามวัตถุประสงค์ที่ระบุไว้ของโครงการนี้หรือไม่
  • การเปลี่ยนแปลงนี้มีผลภายในสถาปัตยกรรมที่มีอยู่ของโปรเจ็กต์หรือไม่
  • การเปลี่ยนแปลงนี้ทำให้เกิดข้อบกพร่องในการออกแบบที่อาจก่อให้เกิดปัญหาในอนาคตหรือไม่
  • การเปลี่ยนแปลงนี้เป็นไปตามแนวทางปฏิบัติที่ดีที่สุด ที่กำหนดไว้สำหรับโปรเจ็กต์นี้หรือไม่
  • การเปลี่ยนแปลงนี้เป็นวิธีที่ดีในการดำเนินการตามฟังก์ชันที่อธิบายใช่ไหม
  • การเปลี่ยนแปลงนี้ก่อให้เกิดความเสี่ยงด้านความปลอดภัยหรือความไม่เสถียร หรือทำให้แอปที่มีอยู่เสียหายหรือไม่

สำหรับข้อมูลเพิ่มเติมเกี่ยวกับการตรวจสอบการเปลี่ยนแปลง โปรดดูส่วนการตรวจสอบการเปลี่ยนแปลงในเอกสารประกอบของ Gerrit

หากคุณได้รับมอบหมายให้เป็นผู้ตรวจสอบการเปลี่ยนแปลง คุณควรได้รับอีเมลที่มีลิงก์ไปยังการเปลี่ยนแปลงดังกล่าว โปรดทำตามขั้นตอนต่อไปนี้เพื่อตรวจสอบและตรวจทานการเปลี่ยนแปลง

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

  2. อ่านคำอธิบายการเปลี่ยนแปลงเพื่อทำความเข้าใจวัตถุประสงค์ของการเปลี่ยนแปลง

  3. คลิกไฟล์เพื่อตรวจสอบการเปลี่ยนแปลงในไฟล์ ไฟล์จะเปิดขึ้นในหน้าจอเปรียบเทียบ 2 แผงโดยมีไฟล์ที่มีอยู่ทางด้านซ้ายและไฟล์ที่มีการเปลี่ยนแปลงที่เสนอทางด้านขวา วิธีตรวจสอบไฟล์

    1. ใช้เมาส์ไฮไลต์ (คลิก + ลาก) ส่วนของโค้ดที่คุณมีความคิดเห็นหรือคำถาม ข้อความ "กด c เพื่อแสดงความคิดเห็น" จะปรากฏขึ้น

    2. กด c ช่องความคิดเห็นฉบับร่างจะปรากฏขึ้น

    3. ป้อนความคิดเห็นหรือคำถาม แล้วคลิกบันทึก

  4. คลิกหมายเลขที่ไม่ซ้ำกันของการเปลี่ยนแปลง (ด้านซ้ายบนของหน้าจอ) เพื่อกลับไปที่หน้าจอข้อมูลการเปลี่ยนแปลง

  5. ตรวจสอบแต่ละไฟล์ในการเปลี่ยนแปลงต่อไป

  6. เมื่อตรวจสอบไฟล์ทั้งหมดในการเปลี่ยนแปลงเสร็จแล้ว ให้กลับไปที่หน้าจอข้อมูลการเปลี่ยนแปลง แล้วคลิกตอบกลับ หน้าจอ "ส่งคะแนนโหวต" จะปรากฏขึ้น

  7. เลือกคะแนนใดคะแนนหนึ่งต่อไปนี้ สำหรับการเปลี่ยนแปลง

    • +1 ดูคล้ายฉันดี แต่คนอื่นต้องอนุมัติด้วย
    • 0 ไม่มีคะแนน

    • -1 ฉันไม่ต้องการให้ส่งอีเมลนี้ตามที่เป็นอยู่ มีสิ่งที่ต้องแก้ไข

    • -2 ไม่ควรส่งการเปลี่ยนแปลงนี้

  8. คลิกตอบกลับเพื่อส่งความคิดเห็นและคะแนนให้นักพัฒนาซอฟต์แวร์ที่ทำการเปลี่ยนแปลง

หากมีปัญหาที่ต้องแก้ไข นักพัฒนาซอฟต์แวร์ได้แก้ปัญหานั้นและขอให้คุณตรวจสอบการแก้ไขดังกล่าว ไม่เช่นนั้น จะถือว่าคุณตรวจสอบการเปลี่ยนแปลงเสร็จแล้ว