วิธีการตรวจสอบเพื่อนกับ GitHub คำขอดึง?


12

เรากำลังย้ายจาก Bitbucket ไปเป็น GitHub และสิ่งหนึ่งที่เรากำลังดิ้นรนคือความคิดเห็นเกี่ยวกับโค้ดเพียร์ที่ทำงานได้อย่างราบรื่นบน Bitbucket เช่นนี้:

  1. ผู้เขียนเปิดคำขอดึง(GitHub: เหมือนกัน)
  2. ผู้เขียนเพิ่มเพื่อนร่วมงานของเขา / เธอเป็นผู้ตรวจสอบ(GitHub: ดิ้นรนที่นี่กับผู้รับมอบหมายหลายคน )
  3. ผู้ตรวจสอบทั้ง:
    1. อนุมัติ PR ด้วยเครื่องหมายสีเขียว(GitHub: ??)
    2. เพิ่มความคิดเห็น(GitHub: เหมือนกัน)
    3. สร้างงานที่มีน้ำหนักเบา(GitHub: เรียงลำดับที่คล้ายกันหาก- [ ]มีการใช้ไวยากรณ์ในคำอธิบาย PR; น่าเสียดายที่มันไม่ทำงานสำหรับงาน)
  4. มีรายการของ PRs ที่ฉันสามารถดูได้อย่างรวดเร็วซึ่งมีการตรวจทานและตกลงที่จะรวมและที่ต้องการความสนใจเพิ่มเติม(GitHub: ??)

ฉันควรชี้ให้เห็นว่าเราต้องการหลีกเลี่ยงเครื่องมือตรวจสอบโค้ดของบุคคลที่สามหากเป็นไปได้ทั้งหมดและต้องการที่จะอยู่ใน Vanilla GitHub ด้วยวิธีแก้ปัญหาบางอย่าง


1
เสียงเหมือนคุณอาจเปลี่ยนก่อนเวลาอันควร ทำไมต้องเปลี่ยนโดยเฉพาะอย่างยิ่งถ้าสิ่งใหม่ไม่มีคุณสมบัติทั้งหมดที่คุณต้องการ
พี่เลี้ยง

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

คำตอบ:


6

จากสิ่งที่ฉันได้เห็นขั้นตอนเหล่านั้นส่วนใหญ่จะทำใน Github โดยการประชุมและไม่ใช่โดยกระบวนการที่ Github ให้มาอย่างเป็นทางการ

นายจ้างของฉันใช้ Github ฉันทำโครงการโอเพนซอร์ซขนาดเล็กจำนวนมากและมีส่วนร่วมในโครงการโอเพ่นซอร์สอื่น ๆ เป็นครั้งคราว

นี่คือวิธีที่ฉันมักจะเห็นมันทำ:

ผู้เขียนเพิ่มเพื่อนร่วมงานของเขา / เธอเป็นผู้ตรวจสอบ:

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

ดูเหมือนว่าโครงการโอเพ่นซอร์สจะมีลำดับชั้นคร่าวๆ - บางทีการประชุมของพวกเขาอาจจะรวมหลังจากผู้สนับสนุน "หลัก" ได้ให้การตกลง

ที่ร้านค้าที่ฉันทำงานอยู่ในปัจจุบันเราผสานหลังจากนักพัฒนาครึ่งโหลคนใดคนหนึ่งในทีมได้ให้ความเห็นชอบแล้ว

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

การอนุมัติจากผู้ตรวจสอบ:

โดยทั่วไปแล้วการอนุมัติจะแสดงขึ้นโดยการแสดงความคิดเห็นในคำขอดึงที่มีคำว่า "+1" หรือ "lgtm" (ดูดีสำหรับฉัน)

งานที่มีน้ำหนักเบา:

ฉันเคยใช้ช่องทำเครื่องหมายเช่นกัน แต่โดยส่วนใหญ่แล้วทุกความคิดเห็นในคำขอดึงถือว่าเป็น "งาน" โดยปริยายซึ่งแก้ไขได้โดย:

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

ดูภาพรวมสิ่งที่ได้รับอนุมัติและสิ่งที่ยังต้องการการตรวจสอบ:

ฉันใช้ส่วนขยายLooks Good To Meสำหรับ Chrome ซึ่งให้มุมมองดังกล่าวจากหน้าจอดึงคำขอ มุมมองรายการคำขอดึงดูเหมือนว่าจะเสียโดยการเปลี่ยนแปลง Github ล่าสุด

โดยการใช้ไซต์ของเรา หมายความว่าคุณได้อ่านและทำความเข้าใจนโยบายคุกกี้และนโยบายความเป็นส่วนตัวของเราแล้ว
Licensed under cc by-sa 3.0 with attribution required.