ผู้ดูแล GitHub ควรเขียนของผู้เขียนในคำขอดึง?


44

ฉันไม่ใช่โปรแกรมเมอร์โดยอาชีพ แต่ฉันเขียนโค้ดและใช้ github บ้าง ฉันเจอสิ่งที่ฉันพบว่าเป็นสถานการณ์ที่น่าแปลกใจ ฉันคุ้นเคยกับคอมไพล์มาก

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

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

เท่าที่ฉันสามารถบอกวิธีเดียวที่จะทำเช่นนั้นโดยเฉพาะอย่างยิ่งใช้ rebase, แก้ไข, หรือเขียนประวัติศาสตร์อื่น ๆ เพื่อลบผู้เขียนต้นฉบับ

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

เป็นเรื่องปกติหรือไม่ ฉันควรพูดอะไรเกี่ยวกับเรื่องนี้หรือไม่?

แก้ไข: ความรู้สึกทั่วไปน่าจะเป็นที่ฉันควรไปถามดังนั้นฉันจะทำแค่นั้นเมื่อเช้านี้

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

อัปเดต ดูเหมือนว่าคำตอบคือผู้เขียนเก็บรักษาสาขาการพัฒนาและไม่ต้องการผสานจากสาขาหลักของเขาเข้ามา เขาเขียนความมุ่งมั่นของฉันอีกครั้งเพื่อหลีกเลี่ยงการรวม ฉันไม่ได้กังวลเกี่ยวกับสาขาดั้งเดิม b / c git มากมายที่มีประสิทธิภาพในการเลือกเชอร์รี่, ลดราคา, และผสานกระทำตามที่ต้องการ

เรื่องนี้เป็นเรื่องปกติใน GitHub?
ฉันควรจะติดต่อผู้ดูแลโครงการเพื่อขอให้สาขาใดใช้แพทช์กับ?


7
+1 สำหรับการสร้างคำถามเกี่ยวกับจริยธรรมเกี่ยวกับการเข้ารหัส :) สนใจที่จะทราบว่านี่เป็นพฤติกรรมเริ่มต้นหรือไม่ดูเหมือนว่าจะมีงานพิเศษเพิ่มอีกเล็กน้อยสำหรับผู้ดูแลที่จะทำสิ่งนี้ หรือสิ่งนี้จะเกิดขึ้นหากผู้ดูแลปรับเปลี่ยนการกระทำของคุณเล็กน้อยหลังจากยอมรับคำขอการดึงของคุณ?
Sunil D.

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

4
บางทีผู้ดูแลระบบใช้การเปลี่ยนแปลงของคุณด้วยตนเองแทนที่จะรวมเข้าด้วยกัน ฉันแนะนำให้ถามผู้ดูแลเกี่ยวกับเรื่องนี้ (โดยไม่กล่าวหาว่าเขา / เธอไม่ซื่อสัตย์)
Keith Thompson

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

2
คุณไม่ได้บอกว่าการแก้ไขนั้นมีขนาดใหญ่เพียงใด (บรรทัดของโค้ด) หรือวิธีการให้สิทธิ์ในการใช้งานของรหัส แต่เนื้อหานั้นอาจเป็นการละเมิดลิขสิทธิ์ของคุณ
Jaydee

คำตอบ:


20

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

  • หากพวกเขาต้องการแก้ไขการเปลี่ยนแปลงของคุณก่อนที่จะนำไปใช้กับสาขาหลักพวกเขาสามารถสร้างสาขาสำหรับการเปลี่ยนแปลงของคุณได้อย่างง่ายดาย พวกเขาสามารถเพิ่มความมุ่งมั่นของตัวเองหลังจากคุณแล้วรวมสาขาใน
  • git rebase masterหากคำขอดึงของคุณไม่ได้ขึ้นอยู่กับรุ่นล่าสุดของสาขาหลักของพวกเขาแล้วพวกเขาก็สามารถออก หากมีข้อขัดแย้งพวกเขาสามารถเลือกที่จะแก้ไขข้อขัดแย้งด้วยตนเอง (โดยไม่ต้องเปลี่ยนผู้เขียน) หรือให้โอกาสคุณในการแก้ไข

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


6

คุณได้ทิ้งรายละเอียดสำคัญไว้ที่นี่แล้ว

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

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

คุณควรตรวจสอบการยอมรับและอัปเดตคำถามของคุณด้วยสิ่งที่คุณค้นพบ


1
ฉันเพิ่งไปตรวจสอบสิ่งนี้ ไม่มีการเปลี่ยนแปลงแก้ไขของฉัน มีการเปลี่ยนแปลง 1 บรรทัดก่อนหน้านี้และไม่เกี่ยวข้องกับโปรแกรมแก้ไขของฉันที่เพิ่มในเวลาเดียวกัน
user1585512

3

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


12
git cherry-pickแล้วพวกเขาก็ควรจะใช้
svick

3

หากต้องการตอบคำถามที่อัปเดตของคุณ:

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

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

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

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