บน GitHub อะไรคือความแตกต่างระหว่างผู้ตรวจสอบและผู้รับมอบหมาย

ฟีเจอร์ที่เพิ่มเข้ามาในวันที่ 7 ธันวาคม 2016 ประกาศบนบล็อก GitHub แนะนำตัวเลือกเพื่อเพิ่มผู้ตรวจสอบในคำขอดึง

ตอนนี้คุณสามารถขอความเห็นอย่างชัดเจนจากผู้ทำงานร่วมกันทำให้ง่ายขึ้นในการระบุผู้ที่คุณต้องการตรวจสอบคำขอดึงของคุณ

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

อย่างไรก็ตามการตั้งค่าผู้ตรวจทานอย่างชัดเจนสำหรับ PR ได้ทำไปแล้วโดยกำหนดคน ( ตัวเลือกผู้รับมอบหมาย )

ขณะนี้ทั้งสองตัวเลือกมีบทบาทของตัวเลือกอย่างไรบ้างเนื่องจากตัวเลือกทั้งสองมีเป้าหมายเดียวกัน

แก้ไข:

หลังจากพูดคุยกับผู้ดูแลระบบ OSS หลายคนผู้ตรวจสอบจะถูกนิยามว่าเป็นสิ่งที่ควรจะเป็น: การตรวจสอบ (รหัสของใครบางคน) และ "ผู้รับมอบหมาย" มีคำจำกัดความที่ชัดเจนกว่าด้านล่าง

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

สำหรับ "ผู้รับมอบหมาย" : ขึ้นอยู่กับทีม / ผู้ดูแลโครงการว่ามันหมายถึงอะไรและไม่มีคำจำกัดความที่เข้มงวด อาจเป็นเครื่องเปิด PR หรือผู้รับผิดชอบพื้นที่นั้น (ผู้ที่จะยอมรับการประชาสัมพันธ์หลังจากการตรวจสอบเสร็จสิ้นหรือเพิ่งปิด) ไม่ขึ้นอยู่กับ GitHub ที่จะกำหนดสิ่งที่เปิดทิ้งไว้สำหรับผู้ดูแลโครงการที่เหมาะสมที่สุดสำหรับโครงการของพวกเขา

คำตอบก่อนหน้า:

ตกลงฉันจะไปข้างหน้าและตอบคำถามของฉันเอง

สำหรับ PR ของผู้ใช้ที่มีการเข้าถึงแบบเขียน:ผู้รับมอบหมายจะเป็นบุคคลเดียวกันกับที่เปิด PR และผู้ตรวจสอบจะแทนที่ฟังก์ชันผู้รับโอนสิทธิ์เดิม (รหัสตรวจสอบ) ซึ่งเป็นการเลือกผู้รับมอบหมายนี้

สำหรับ PR ของผู้ใช้ที่ไม่มีการเข้าถึงเพื่อเขียน (ผู้มีส่วนร่วมภายนอก):คนที่มีสิทธิ์การเข้าถึงจะกำหนดตัวเอง (หรือสมาชิกที่มีสิทธิ์ในการเขียนอื่น ๆ ) เพื่อตรวจทาน PR (ผู้ตรวจทาน) ผู้รับมอบหมายว่างเปล่า

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

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

ตามเอกสาร GitHubอย่างเป็นทางการผู้ได้รับมอบหมายเป็นบุคคลที่ทำงานเกี่ยวกับปัญหาเฉพาะและดึงคำขอ บางครั้งมันสับสนในฐานะผู้ตรวจสอบ มันมีความหมายจริงที่จะใช้กับปัญหามากกว่าคำขอดึงเพื่อให้เมื่อเราได้รับปัญหาเราสามารถมอบหมายให้ใครสักคนแก้ไขได้ ในคำขอดึงข้อมูลผู้รับหมายถึงบุคคลที่รับผิดชอบการรวมคำขอดึงข้อมูลหลังจากได้รับความคิดเห็นและเปลี่ยนคำขอจากผู้ดูแลอื่น ๆ

ตามคำตอบที่ยอมรับ ใช่ "ผู้รับมอบหมาย" มีคำจำกัดความที่หลวมและสามารถใช้แตกต่างกันเพื่อให้เหมาะกับความต้องการของทีม

ในทีมงานของเรา 8 devs ใน PRs ส่วนใหญ่เรามีผู้ตรวจสอบ 1 คนซึ่งแนะนำการเปลี่ยนแปลงและอนุมัติการประชาสัมพันธ์ในท้ายที่สุด ในระหว่างขั้นตอนการตรวจสอบ "ผู้รับมอบหมาย" คือบุคคลที่เปิดการประชาสัมพันธ์ ในภายหลังหากมีการรับ PR จากผู้พัฒนารายอื่นจะมีการเพิ่ม "ผู้รับมอบหมาย" ใหม่ เมื่อ PR ได้รับการอนุมัติและพร้อมสำหรับ QA หรือการผสานโดยตรงจะมีการเพิ่ม "ผู้รับมอบหมาย" QA ใหม่ รายการนี้ "ผู้รับมอบหมาย" จะเติบโต

เราใช้ "ผู้รับมอบหมาย" เพื่อกำหนดบุคคลต่อไปนี้ให้เรียกรวมกัน:

1. ดึงผู้เขียนคำขอ
2. ผู้เขียนทำงานกับข้อเสนอแนะการเปลี่ยนแปลง PR (มักจะเหมือนกับ 1)
3. บุคคลที่เกี่ยวข้องกับการประกันคุณภาพ
4. ผู้รับผิดชอบในการรวม (มักจะเหมือนกับ 2 หรือ 3)

การใช้ "ผู้รับมอบหมาย" จะช่วยให้ค้นหาตำแหน่ง PR ในอนาคตได้อย่างง่ายดาย หนึ่งในโครงการของฉันมี> 3000 PRs

is:open is:pr author:raya-dumas

is:closed is:pr assignee:raya-dumas

หรือเพียงแค่author:raya-dumasค้นหารายการทั้งหมดที่ผู้สร้างสร้างขึ้น (ฉบับ, PRs)

และคำค้นหาที่คล้ายกันอื่น ๆ เพื่อทำให้กระบวนการค้นหาง่ายขึ้น "เหตุการณ์สำคัญ" มีประโยชน์มากในการใช้งานและช่วยให้การค้นหา PR ง่ายขึ้น

ก่อนที่ GitHub จะมีฟิลด์ผู้รับเท่านั้นและไม่มีฟิลด์ผู้ตรวจสอบ ไม่มีความแตกต่างในตอนนั้นดังนั้นผู้รับโอนมักใช้เป็นฟิลด์ผู้ตรวจทานมากที่สุด

แต่ใช้สิ่งที่เหมาะสมกับโครงการของคุณ