เราควรใช้ Reject หรือ Wait For Author ใน TFS เมื่อมีสิ่งที่ต้องแก้ไขหรือไม่?


13

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

ป้อนคำอธิบายรูปภาพที่นี่

คำตอบ:


15

ตาม Microsoft ในรหัสตรวจทานพร้อมคำขอดึง: ลงคะแนนการเปลี่ยนแปลงวัตถุประสงค์ที่แนะนำของแต่ละประเภทการอนุมัติคือ:

อนุมัติข้อเสนอแนะ : เห็นด้วยกับคำขอดึง แต่ให้คำแนะนำเพิ่มเติมเพื่อปรับปรุงรหัส

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

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

ดังนั้นฉันWaiting for Authorหมายความว่าคุณคิดว่าผู้เขียนเมาในวิธีการแก้ปัญหาของเขา / เธอ แต่รหัสของเขา / เธอสามารถแลกได้หากพวกเขานำความคิดเห็นของคุณไปสู่หัวใจ

และRejectedหมายความว่าคุณไม่มีทางที่จะยอมรับการเปลี่ยนแปลงเช่นนี้ไม่ว่าจะเขียนรหัสได้ดีเพียงใด

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


1
เกี่ยวกับสิ่งนี้: "ผู้เขียนควรแจ้งให้คุณทราบเมื่อคุณควรตรวจสอบรหัสอีกครั้งหลังจากที่พวกเขาได้แจ้งข้อกังวลของคุณแล้ว" - มีกลไกสำหรับผู้เขียนเพื่อแจ้งเตือนผู้ตรวจทานว่า PR พร้อมสำหรับการทบทวนอีกครั้งหรือไม่?
Robert Sim

@RobertSim - เราใช้เป็นตัวเลือก 'Ping' ภายใต้ผู้ตรวจสอบเพื่อส่งข้อความขอให้ตรวจทานอีกครั้ง
killercowuk

วิธีอื่นในการดู: Waiting for Author= ฉันต้องการคุณลักษณะนี้ แต่คุณเขียนผิดและฉันจะไม่อนุมัติจนกว่าคุณจะแก้ไขตามที่ฉันแนะนำ และRejected= ฉันไม่ต้องการคุณลักษณะนี้ที่คุณขอให้ฉันตรวจสอบ
goku_da_master
โดยการใช้ไซต์ของเรา หมายความว่าคุณได้อ่านและทำความเข้าใจนโยบายคุกกี้และนโยบายความเป็นส่วนตัวของเราแล้ว
Licensed under cc by-sa 3.0 with attribution required.