รีวิวรหัส TFS - แสดงไฟล์ที่อัปเดตเพื่อตอบสนองความคิดเห็น


128

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

ดังนั้นกระบวนการจะเป็นดังนี้:

  1. บุคคลที่ 1 ร้องขอการตรวจสอบโค้ด
  2. บุคคลที่ 2 เพิ่มความคิดเห็นและเลือก "ต้องการงาน"
  3. บุคคลที่ 1 ทำการเปลี่ยนแปลงที่จำเป็น

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

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

มีเว็บไซต์มากมายที่แสดงวิธีใช้ฟังก์ชันพื้นฐานของการตรวจทานโค้ด แต่มีไซต์ใดบ้างที่แสดงแนวทางปฏิบัติที่ดีที่สุด ตัวอย่างเช่นใครควรคลิกช่องทำเครื่องหมายข้างไฟล์


+1 สำหรับ "มีไซต์ใดบ้างที่แสดงแนวทางปฏิบัติที่ดีที่สุด" โดยปกติแล้ว ALM Rangers จะเป็น "goto guys" ของฉันสำหรับการใช้งานเชิงลึก แต่ข้อมูลเดียวที่ฉันหาได้ไม่ตอบคำถามของคุณ ฉันชอบที่จะเห็นแนวทางและสถานการณ์ในการใช้ Code Review
DaveShaw

คำตอบ:


165

ดังนั้นกระบวนการจะเป็นดังนี้:

  1. บุคคลที่ 1 ร้องขอการตรวจสอบโค้ด
  2. บุคคลที่ 2 เพิ่มความคิดเห็นและเลือก "ต้องการงาน"
  3. บุคคลที่ 1 ทำการเปลี่ยนแปลงที่จำเป็น
  4. บุคคลที่ 1 อัปเดตชุดชั้นวางที่เกี่ยวข้องกับการตรวจสอบโค้ด
  5. บุคคลที่ 1 เพิ่มความคิดเห็นเพื่อดำเนินการสนทนาต่อ
  6. ทำซ้ำขั้นตอนที่ 2 - 5 จนกว่าจะยอมรับ

ขั้นตอนที่จำเป็นในการอัปเดตชุดชั้นวางที่เกี่ยวข้องกับการตรวจสอบมีดังนี้

  1. จากบานหน้าต่าง "การตรวจสอบโค้ด" ให้เลือกลิงก์ "ดูชุดชั้นวาง"
  2. จากบานหน้าต่าง "รายละเอียดชั้นวาง" ไฮไลต์แล้วคัดลอกชื่อชุดชั้นวาง
  3. ไปที่บานหน้าต่าง "การเปลี่ยนแปลงที่รอดำเนินการ" คลิกที่ "Shelve" และวางชื่อชุดชั้นวาง
  4. กดปุ่มใช่บนกล่องโต้ตอบการตรวจสอบแทนที่ชุดชั้นวาง
  5. ขณะนี้ผู้ตรวจสอบสามารถดูไฟล์ที่อัปเดตแล้วและการอภิปรายเกี่ยวกับบทวิจารณ์สามารถดำเนินต่อ

ฉันได้รวมภาพหน้าจอไว้ด้วยเพราะฉันพบว่ามันช่วยให้ความกระจ่างในสิ่งต่างๆ


1) จากบานหน้าต่าง "การตรวจสอบโค้ด" ให้เลือกลิงก์ "ดูชุดชั้นวาง" ดังที่แสดงไว้ที่นี่:

ใส่คำอธิบายภาพที่นี่


2) จากบานหน้าต่าง "รายละเอียดชั้นวาง" ไฮไลต์และคัดลอกชื่อชุดชั้นวางตามที่แสดงที่นี่:

ใส่คำอธิบายภาพที่นี่


3) ไปที่บานหน้าต่าง "การเปลี่ยนแปลงที่รอดำเนินการ" คลิกที่ "Shelve" และวางชื่อชุดชั้นวางเช่น:

ใส่คำอธิบายภาพที่นี่


4) กดปุ่มใช่บนกล่องโต้ตอบการตรวจสอบการแทนที่ชั้นวาง:

ใส่คำอธิบายภาพที่นี่


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

15
การเปลี่ยนชุดชั้นวางไม่ได้สัมผัสเฉพาะความคิดเห็นของไฟล์ ทีมของเราใช้วิธีนี้สำเร็จและเราจะไม่หลุดความคิดเห็นใด ๆ
Chad Dienhart

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

2
ฉันมักจะไม่ชอบพูดซ้ำ แต่ประเด็นของ @ Chin ด้านบนทำให้คำตอบนี้ไม่สามารถปฏิเสธได้อย่างสมบูรณ์ ใช่คุณสามารถอัปเดตชุดชั้นวางของคุณได้อย่างง่ายดาย แต่การทำเช่นนั้นทำให้ความคิดเห็นในการตรวจสอบก่อนหน้านี้ทั้งหมดชี้ไปที่ข้อความที่ไม่ถูกต้อง (เนื่องจากเห็นได้ชัดว่ามีการแนบความคิดเห็นของบทวิจารณ์เข้ากับการหักล้างสัมบูรณ์จากจุดเริ่มต้นของไฟล์)
Michael Sorens

7
คู่มือเกินไปและไม่ใช่สิ่งที่เหมาะ TFS หรือ Visual Studio ควรมีบางอย่างเพื่อเปรียบเทียบการเปลี่ยนแปลงระหว่างบทวิจารณ์ที่ต่อเนื่องกันเพื่อให้ผู้ตรวจทานสามารถเห็นการเปลี่ยนแปลงที่เกิดขึ้นตั้งแต่การตรวจสอบครั้งล่าสุด ไม่มีอะไรแบบนี้? อาจจะเป็นปลั๊กอินหรืออะไรก็ได้ ...
Iúri dos Anjos

6

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


3

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

เมื่อเร็ว ๆ นี้ทีม TFS ได้เริ่มขั้นตอนการวางแผนสำหรับวิธีแก้ปัญหาสำหรับคำขอใช้บริการผู้ใช้เก่ามากเพื่อเปิดใช้งานการอัปเดตการตรวจสอบโค้ดใน ขณะนี้ได้ย้ายไปที่ developercommunity.visualstudioซึ่งควรแก้ปัญหานี้อย่างหรูหราด้วยการทำให้ Code Reviews มีการทำซ้ำ


1
ยิ่งใหญ่ ฉันจะรอคอยเช่นกัน ฉันยินดีที่จะแก้ปัญหาที่แท้จริง โพสต์ของฉันเป็นเพียงวิธีการแก้ไขปัญหา
Chad Dienhart

หวังว่ามันจะไม่เหมือนคำขอดึง azure DevOps พวกเขาเป็นฝันร้ายที่เป็นสแปม
StingyJack

" อินสแตนซ์ UserVoice นี้ไม่สามารถใช้ได้อีก. "
รัฟฟิน

1
ขอบคุณที่ติดตาม @ruffin อัปเดตคำตอบด้วยลิงก์ใหม่ ปัญหาถูกย้ายจาก UserVoice ไปยังไซต์ใหม่ คุณสามารถตรวจสอบอีกครั้งว่าเป็นปัญหาเดียวกันทุกประการโดย URL ที่กล่าวถึงในหน้าใหม่ใน "ย้ายข้อมูลจาก"
julealgon

0

คุณต้องทำสิ่งนี้ด้วยบทวิจารณ์ที่แตกต่างกันสองรายการ แต่ยังมีวิธีการเก็บประวัติด้วยการทบทวนครั้งที่สอง สิ่งที่คุณต้องมีคืองาน

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

  1. สร้างงาน 1
  2. ก่อนตรวจสอบใน changeset1 ให้เพิ่ม task1 เป็นไอเท็มงานที่เกี่ยวข้อง
  3. ตรวจสอบการเปลี่ยนแปลงด้วยรายการงานหนึ่งรายการที่เกี่ยวข้องและขอให้ตรวจสอบชุดการเปลี่ยนแปลงนี้
  4. สร้างงาน 2
  5. ก่อนตรวจสอบใน changeset2 ให้เพิ่มงานทั้งสองเป็นรายการงานที่เกี่ยวข้อง
  6. ตรวจสอบการเปลี่ยนแปลงกับสองรายการงานที่เกี่ยวข้องและขอให้ตรวจสอบชุดการเปลี่ยนแปลงนี้

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

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