คำถามติดแท็ก pull-requests

6
ทำไมคอมไพล์สควอชจึงส่งคำขอดึง?
ทำไม Github ที่ร้ายแรงทุกคนที่ฉันสั่งซื้อซ้ำทำให้อยากให้ฉันสควอชให้คำมั่นสัญญาของฉัน? ฉันคิดว่าบันทึกการคอมไพล์อยู่ที่นั่นเพื่อให้คุณสามารถตรวจสอบประวัติทั้งหมดของคุณและดูว่าการเปลี่ยนแปลงเกิดขึ้นที่ไหน แต่การบีบมันดึงมันออกมาจากประวัติศาสตร์และทำให้มันกลายเป็นหนึ่งเดียว ประเด็นคืออะไร? สิ่งนี้ก็ดูเหมือนว่าจะไปกับ "กระทำเร็วและกระทำบ่อย" มนต์

4
วิธีการจัดการสิ่งที่ต้องทำในคำขอดึง?
คำถามนี้ถูกย้ายจากการแลกเปลี่ยนการประกันคุณภาพซอฟต์แวร์และการทดสอบสแต็คเพราะสามารถตอบได้ใน Software Engineering Stack Exchange ย้ายไป เมื่อปีที่แล้ว เมื่อฉันตรวจทานการเปลี่ยนแปลงในคำขอดึงบางครั้งฉันสะดุดความคิดเห็นด้วยหมายเหตุ "สิ่งที่ต้องทำ" ซึ่งอาจมีสาเหตุที่แตกต่างกันในกรณีของเราส่วนใหญ่เนื่องจาก: โซลูชันที่ใช้ในการแก้ปัญหานั้นสามารถปรับปรุงได้ แต่จะต้องใช้เวลามากขึ้นในการลงทุน ผู้เขียนเลือกวิธีแก้ปัญหาที่รวดเร็วกว่า แต่ใส่ความเห็นว่าอาจมีตัวเลือกที่ดีกว่า มีรหัสชั่วคราวเพื่อแก้ไขข้อบกพร่องที่มีอยู่ซึ่งควรได้รับการแก้ไขในไม่ช้า การรู้ว่าTODOโดยทั่วไปแล้วจะอยู่ใน codebase ตลอดอายุการใช้งานของ codebase ฉันจะตอบสนองต่อพวกเขาอย่างไรในคำขอดึง ฉันจะขอให้หลีกเลี่ยงอย่างสุภาพได้อย่างไรหรือหากมีเหตุผลจริง ๆ ฉันจะแน่ใจได้อย่างไรว่าผู้เขียนประชาสัมพันธ์จะติดตามมันในอนาคต

1
คำขอดึง GitHub ของฉันถูกรวมการประชุมในขั้นตอนนี้คืออะไร?
ฉันคดเคี้ยวโครงการ Github ที่ทำการเปลี่ยนแปลงเล็ก ๆ และส่งคำขอดึงไปยังผู้ดูแลเดิมที่ได้ดึงมันใน. Merged pull request #11 from my_username/masterตอนสุดท้ายกระทำมี นี่เป็นครั้งแรกที่ฉันทำสิ่งนี้ดังนั้นฉันไม่แน่ใจว่ามารยาทในตอนนี้คือ: ฉันทำgit pull upstream masterแล้วgit push origin masterและตอนนี้ความมุ่งมั่นล่าสุดของที่เก็บของฉันเองMerged pull request #11 from my_username/masterซึ่งอ่านแล้วรู้สึกแปลกสำหรับฉัน นี่เป็นวิธีที่ผู้คนมักจะทำหรือไม่ฉันต้องทำอะไรเพื่อ "ล้างประวัติ" หรืออะไรบางอย่าง? หมายเหตุ: เนื่องจากนี่เป็นการเปลี่ยนแปลงเอกสารเล็กน้อยฉันไม่ได้สร้างสาขาเลยฉันเพิ่งทำการเปลี่ยนแปลงในmasterสาขาของฉันและส่ง pull req ดังนั้นจึงไม่มีการล้างข้อมูลในส่วนนั้น

4
รหัสตรวจสอบเวิร์กโฟลว์ที่ผู้เขียนคำขอดึงจะต้องผสาน
หลาย ๆ ทีมที่ บริษัท ของฉันฝึกฝนเวิร์กโฟลว์การตรวจสอบโค้ดที่ฉันไม่เคยเห็นมาก่อน ฉันพยายามเข้าใจความคิดเบื้องหลังด้วยความคิดที่ว่ามีคุณค่าในการทำให้ บริษัท ทั้งหมดสอดคล้องกัน (ฉันมีส่วนร่วมกับรหัสฐานหลายและถูกเพิ่มขึ้นตามความแตกต่างในอดีต) ผู้เขียนโค้ดส่งคำขอการดึง ผู้ตรวจสอบตรวจสอบรหัส หากผู้ตรวจสอบอนุมัติพวกเขาจะแสดงความคิดเห็นตามบรรทัดของ "ดูดีรู้สึกอิสระที่จะผสาน" หากผู้ตรวจสอบมีข้อกังวลพวกเขาแสดงความคิดเห็นเช่น "โปรดแก้ไขปัญหาเล็กน้อย X และ Y จากนั้นรวม" (สำหรับการเปลี่ยนแปลงที่สำคัญกลับไปที่ขั้นตอนที่ 2) ผู้สร้างรหัสทำการเปลี่ยนแปลงหากจำเป็นจากนั้นรวมคำขอดึงของเขาหรือเธอเอง ฉันมีข้อกังวลดังต่อไปนี้: ในกรณีที่ได้รับการอนุมัติในขั้นตอนที่ 3 เวิร์กโฟลว์นี้จะสร้างการเดินทางไปกลับที่ไม่จำเป็นสำหรับผู้เขียนคำขอดึง ผู้ตรวจทานซึ่งกำลังดูรหัสอยู่แล้วสามารถรวมมันได้ทันที ในกรณีที่มีการร้องขอการเปลี่ยนแปลงในขั้นตอนที่ 3 หน่วยงานที่จะรวมคำขอดึงข้อมูลจะอยู่กับผู้แต่งของ PR เท่านั้น ไม่มีใครนอกจากผู้เขียนจะดูการเปลี่ยนแปลงก่อนที่จะรวม อะไรคือข้อดีหรือข้อเสียของเวิร์กโฟลว์นี้ เวิร์กโฟลว์นี้เป็นเรื่องปกติในทีมวิศวกรรมอื่น ๆ หรือไม่

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

1
เราควรใช้ Reject หรือ Wait For Author ใน TFS เมื่อมีสิ่งที่ต้องแก้ไขหรือไม่?
ใน TFS เมื่อเราใส่ความคิดเห็นเพื่อแก้ไขสิ่งต่างๆในคำขอดึงก่อนที่เราจะยอมรับเราควรทำเครื่องหมายว่าปฏิเสธหรือรอผู้แต่งหรือไม่ ไหนดีกว่ากัน

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

1
ฉันแก้ไขโครงการทั้งหมดของบางคนโดยไม่ตั้งใจ มีวิธีใดที่จะดึงคำขอได้หรือไม่
ฉันพบโปรเจ็กต์ที่ยอดเยี่ยมเกี่ยวกับ GitHub ที่มีคุณสมบัติศูนย์กลางที่มีประโยชน์ แต่หยาบใน "การขัด" ของการจัดการข้อผิดพลาดการบันทึกการกำหนดค่าและการตั้งค่า โครงการไม่มีการเปลี่ยนแปลง 5 ปีและมีรหัสเพียงสองร้อยบรรทัด ยังคงมีประโยชน์มากพอที่จะให้ความสนใจกับนักดูจำนวนที่เหมาะสมและส้อมจำนวนหนึ่ง จำเป็นต้องมีการเพิ่มเฉพาะสำหรับการใช้งานของฉัน แต่ก่อนหน้านั้นฉันทำการล้างข้อมูล จากนั้นฉันก็ได้วิศวกรที่คลั่งไคล้และในช่วงหนึ่งสัปดาห์ที่ฉันจบลงด้วยการเพิ่มระบบการบันทึกการบันทึกจำนวนมากการตั้งค่าอัตโนมัติการกำหนดค่าที่แยกออกจากรหัสและลงในไฟล์การกำหนดค่าภายนอก (และรหัสเพื่ออ่าน) รวมทั้งแก้ไขข้อบกพร่องสองสามเช่นเดียวกับที่ฉันพบพวกเขา ฉันคิดว่าการเปลี่ยนแปลงทั้งหมดของฉันนั้นสมเหตุสมผล / ดีและเป็นสิ่งที่ผู้ชมจำเป็นต้องใช้ แต่มีความมุ่งมั่นจำนวนมากเกือบจะมากที่สุดเท่าที่ repo มีอยู่เดิม (หลีกเลี่ยงตัวเลขเพื่อรักษาข้อมูลทั่วไปนี้) นอกจากนี้การตำหนิ git ยังแสดงให้เห็นว่าฉันได้สัมผัสเกือบทุกบรรทัดของ codebase นี้ (เล็ก!) ฉันไม่ได้ต้องการควบคุมโครงการหรือจำเป็นต้องให้เครดิตกับสิ่งที่ฉันได้ทำไป แต่ให้ทางเลือกที่ฉันต้องการที่จะเห็นการเปลี่ยนแปลงของฉันรวมเพื่อให้ทุกคนสามารถได้รับประโยชน์จากพวกเขามากกว่าที่จะซ่อนอยู่ในทางแยกใน GitHub ที่ไม่รู้จักของฉัน ฉันไม่เคยส่งคำขอดึงมาก่อน แต่ฉันรู้ว่าควรมีขนาดเล็กและง่ายต่อการตรวจสอบ แต่ที่นี่ฉันลงไปและทำการเปลี่ยนแปลงเปลี่ยนแปลงจำนวนมาก ความมุ่งมั่นของฉันนั้นสะอาดมากเมื่อฉันมีประวัติอย่างระมัดระวังตลอด แต่พวกเขาก็ยากที่จะแยกออกเป็นหลายสาขา / ดึงคำขอเพราะส่วนใหญ่จำเป็นต้องสร้างขึ้นมาเอง ตัวอย่างเช่นการกำหนดค่าภายนอกถูกสร้างขึ้นในการเตรียมการล้างข้อมูลและมีการตั้งค่าบางส่วนในการเติมการตั้งค่าเหล่านั้นการบันทึกถูกเปิดใช้งานและกำหนดค่าโดยการตั้งค่าภายนอกที่สร้างขึ้นในการตั้งค่าเป็นต้น ทำสิ่งที่ฉันสามารถทำได้เพื่อให้ยาเม็ดใหญ่โตอร่อยกว่านี้ฉันไม่รู้ว่ามันจะเป็นยังไง ฉันแบ่งออกได้สองสามอย่าง แต่การยกเครื่องครั้งใหญ่ยังคงใหญ่อยู่ ดังนั้นจะทำอย่างไรถ้าพวกเขายกเครื่องโครงการของใครบางคนโดยไม่ตั้งใจ? ฉันควรเรียนรู้บทเรียนของฉันที่จะไม่ทำและเก็บการเปลี่ยนแปลงของตัวเองไว้กับตัวเองหรือไม่? ฉันควรจะดึงคำขอและดูว่าเกิดอะไรขึ้น ฉันควรใช้คำศัพท์อธิบายตัวเองเป็นจำนวนมากหรือไม่? ฉันควรจะนำเสนอมันในบางวิธีหรือไม่?

2
มันจะดีกว่าที่จะเริ่มคำขอดึงหรือดำเนินการผสานท้องถิ่นกระทำกับต้นแบบ?
ตอนนี้ฉันใช้ GitHub มาระยะหนึ่งแล้วฉันมักจะใช้ฟีเจอร์ - สาขาแล้วก็เริ่มคำขอดึงซึ่งตัวฉันเองรวมเข้าด้วยกัน ฉันพบว่ามันช่วยฉันติดตามว่าฉันรวมสาขาที่ไหน แต่เมื่อเร็ว ๆ นี้ฉันได้อ่านมากขึ้นเกี่ยวกับวิธีการทำงานของ Git และฉันก็ตระหนักว่าฉันสามารถใช้การผสานที่มุ่งมั่นในการอ้างถึงเมื่อฉันรวมสาขา ดังนั้นฉันควรทำอย่างไรเมื่อรวมฟีเจอร์บรานช์เข้ากับมาสเตอร์: ทำการผสานคอมมิชชันบนมาสเตอร์จากนั้นดันต้นน้ำหรือดันสาขาท้องถิ่นและเริ่มคำขอดึง? ฉันได้อ่านIntroducing Pull Requests สำหรับทีม 2 คนแล้วรวมคำขอของฉันเองหรือ และขั้นตอนการทำงานกับ 2 คนในโครงการคืออะไรและฉันควรจะเปิดคำขอดึงจากสาขาใน repo อย่างเป็นทางการหรือทางแยกของฉัน? แต่ดูเหมือนว่าไม่มีใครตอบคำถามที่ฉันต้องการ

6
ดึงคำขอสถานที่สำหรับฝึกอบรมรุ่นน้อง
เรามีแนวคิดว่าโค้ดทั้งหมดในคำขอดึงเข้าสู่ต้นแบบควรพร้อมใช้งานจริง สิ่งนี้สมเหตุสมผลและเป็นคำแถลงการณ์ที่ยุติธรรมในความคิดของฉัน แนวคิดที่ว่าเมื่อคุณสร้างงานประชาสัมพันธ์คุณกำลังระบุว่าคุณจะต้องทำสิ่งนี้ให้เป็นปรมาจารย์ แต่ต้องการให้ผู้ตรวจสอบบางคนเห็นด้วยกับคุณ เนื่องจากเราเป็นมนุษย์เท่านั้นเราจึงทำผิดพลาดและหวังว่าผู้ตรวจสอบคนอื่นอาจพบรายการที่การทดสอบหน่วยไม่สามารถหาได้ - การสะกดคำผิด javadocs ไม่ถูกต้อง ฯลฯ แต่เป็นคำขอการดึงที่เราควรให้ความช่วยเหลือ / ฝึกอบรมแก่นักพัฒนาในระดับหนึ่งและถ้าเป็นเช่นนั้นในระดับใด ทุกครั้งที่คุณผลักดันการเปลี่ยนแปลงใหม่ผู้ตรวจสอบจะต้องตรวจทานการเปลี่ยนแปลงของคุณอีกครั้งซึ่งต้องใช้เวลาในการพัฒนาและทำให้เกิดการตรวจสอบการเปลี่ยนแปลงอีกครั้ง ดังนั้นควรมีการฝึกอบรมเท่าไหร่ที่คาดว่าจะได้รับอนุญาตในคำขอดึงข้อมูล? ส่วนหนึ่งของฉันรู้สึกว่ามันแตกต่างกันไปตั้งแต่รุ่นน้องจนถึงรุ่นอาวุโส อย่างไรก็ตามฉันก็รู้สึกว่ามันไม่ควรจะเป็นสถานที่สำหรับการค้นหาปัญหาจำนวนมาก - แม้แต่สำหรับรุ่นน้อง มีใครบ้างที่พยายามดิ้นรนเพื่อให้นักพัฒนาซอฟต์แวร์บรรลุเป้าหมายของ "คำขอดึงของฉันควรจะพร้อมสำหรับการผลิต" หรือไม่?

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

1
เหตุใดจึงไม่สามารถดำเนินการคำขอดึงข้อมูลอย่างจริงจังใน Mercurial
หนึ่งสัปดาห์ที่ผ่านมาฉันอยู่ที่LFNWและขณะที่ฉันกำลังพูดคุยกับ Larry Hastings หลังจากการพูดคุยของเขาเขาพูด (ถอดความ): มีบางอย่างใน Git ที่ทำให้กระบวนการดึงคำขอเป็นไปได้ที่ไม่มีอยู่ใน Mercurial นั่นเป็นเหตุผลที่คำขอการดึงใน BitBucket นั้นไม่ดี (สำหรับบริบทเรากำลังคุยกันถึงความจริงที่ว่า Python กำลังย้ายจาก Mercurial มาเป็น Git เนื่องจากเวิร์กโฟลว์ PR ไม่เหมือนกับเวิร์กโฟลว์ file-bug-then-attach-patch เขากำลังพูดถึงอะไรที่นี่? เราสองคนไม่สามารถสร้างชื่ออะไรก็ได้ ฉันค้นหาเว็บโดยไม่มีโชค
โดยการใช้ไซต์ของเรา หมายความว่าคุณได้อ่านและทำความเข้าใจนโยบายคุกกี้และนโยบายความเป็นส่วนตัวของเราแล้ว
Licensed under cc by-sa 3.0 with attribution required.