แนะนำคำขอดึงสำหรับทีม 2 คน - รวมคำขอของฉันเองหรือ


11

ฉันแนะนำคอมไพล์ให้กับสมาชิกในทีมจูเนียร์ (Co-op)

ตอนนี้พวกเขามีความสะดวกสบายกับพื้นฐานของการเพิ่มกระทำการผลักและดึง

ตอนนี้ฉันต้องการแนะนำพวกเขาเพื่อดึงคำขอและสาขา

หากพวกเขาเริ่มทำคำขอดึงในสาขาฉันควรทำเช่นเดียวกันสำหรับการทำงานอย่างต่อเนื่องของฉัน?
ฉันจะเป็นหนึ่งในการรวมคำขอดึงของพวกเขามาฉันไม่แน่ใจว่ามันจะทำให้รู้สึกมากที่สุดสำหรับฉันที่จะทำงานในสาขา (โดยทั่วไปการปฏิบัติที่ดีฉันรู้ แต่ฉันอยากรู้เกี่ยวกับสถานการณ์เฉพาะ 2 devs กับจูเนียร์หนึ่ง ) และถ้าเป็นเช่นนั้นหมายความว่าฉันจะรวมสาขาของตัวเองเป็นเจ้านาย ฉันจะทำคำขอดึงสำหรับงาน / สาขาของฉันด้วยหรือไม่? โดยทั่วไปเราใช้ขั้นตอนการทำงานของ GitHub คุณสมบัติขั้นต้นสำหรับการเปลี่ยนแปลงเหล่านี้:
https://www.atlassian.com/git/tutorials/comparing-workflows/feature-branch-workflow

มีวัตถุประสงค์ในการใช้คำขอดึงบน repo ของฉันเองหรือไม่ถ้าฉันเป็นนักพัฒนาซอฟต์แวร์คนเดียว? มีประโยชน์ แต่ไม่ค่อยมีความเจาะจง

ขั้นตอนการทำงานกับคน 2 คนในโครงการดูเหมือนจะเป็นเรื่องทั่วไปมากขึ้น

และ

ฉันควรเปิดคำขอดึงจากสาขาใน repo อย่างเป็นทางการหรือทางแยกของฉัน? ดูเหมือนจะเกี่ยวกับส้อม

คำตอบ:


19

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

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

  1. คุณทั้งคู่เรียนรู้ว่าเยาวชนของคุณต้องการเน้นที่กิจกรรมการเรียนรู้ที่ไหน
  2. คุณเรียนรู้ว่าคุณฉลาดกว่าที่ควร

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