Đánh giá mã TFS - Hiển thị các tệp được cập nhật để phản hồi ý kiến


127

Chúng tôi đang bắt đầu sử dụng chức năng xem lại mã được tích hợp sẵn để xem trước VS 2012 và VS 2013. Yêu cầu xem xét và thêm ý kiến ​​có vẻ khá đơn giản. Nếu ai đó thêm ý kiến ​​yêu cầu thay đổi mã, thì người yêu cầu thực hiện những thay đổi này và hiển thị chúng như thế nào?

Vì vậy, quá trình sẽ chảy như thế này:

  1. Người 1 yêu cầu xem lại mã.
  2. Người 2 thêm ý kiến ​​và chọn "Nhu cầu làm việc."
  3. Người 1 thực hiện những thay đổi cần thiết.

Làm thế nào để Người 1 bây giờ hiển thị những thay đổi này cho Người 2? Bạn có thể thêm nhận xét và gửi chúng, nhưng các tệp không thay đổi. Tôi giả sử các tệp từ bộ thay đổi được tạo khi đánh giá ban đầu được yêu cầu. Người 1 có nên đóng đánh giá này và yêu cầu đánh giá thứ hai không?

Tạo đánh giá thứ hai có vẻ không tối ưu, vì bạn mất lịch sử cuộc trò chuyện về lý do tại sao bạn thực hiện các thay đổi.

Có rất nhiều trang web cho thấy cách sử dụng chức năng cơ bản của các đánh giá mã, nhưng có trang web nào cho thấy các thực tiễn tốt nhất không? Ví dụ, ai nên nhấp vào hộp kiểm bên cạnh tệp?


+1 cho "có bất kỳ trang web nào hiển thị các thực tiễn tốt nhất". ALM Rangers thường là "những kẻ goto" của tôi để sử dụng chuyên sâu, nhưng thông tin duy nhất tôi có thể tìm thấy, không trả lời câu hỏi của bạn. Tôi rất thích xem một số suy nghĩ và kịch bản để sử dụng Xem lại mã.
DaveShaw

Câu trả lời:


164

Vì vậy, quá trình sẽ chảy như thế này:

  1. Người 1 yêu cầu xem lại mã.
  2. Người 2 thêm ý kiến ​​và chọn "Nhu cầu làm việc."
  3. Người 1 thực hiện những thay đổi cần thiết.
  4. Người 1 Cập nhật các kệ liên quan đến đánh giá mã
  5. Người 1 thêm ý kiến ​​để tiếp tục thảo luận
  6. Lặp lại các bước 2 - 5 cho đến khi được chấp nhận

Dưới đây là các bước cần thiết để cập nhật kệ liên quan đến đánh giá.

  1. Từ khung "Xem lại mã", chọn liên kết "xem kệ"
  2. Từ phần "Chi tiết kệ" tô sáng và sao chép tên kệ
  3. Điều hướng đến ngăn "Đang chờ thay đổi", nhấp vào "Kệ" và dán tên kệ
  4. Nhấn nút Có trên hộp thoại xác minh thay thế kệ
  5. Bây giờ người đánh giá có thể thấy các tệp đã cập nhật và cuộc thảo luận đánh giá có thể tiếp tục

Tôi đã bao gồm một số ảnh chụp màn hình khi tôi thấy nó giúp làm rõ mọi thứ.


1) Từ khung "Xem lại mã", chọn liên kết "xem kệ" như hiển thị ở đây:

nhập mô tả hình ảnh ở đây


2) Từ phần tô sáng "Chi tiết giá đỡ" và sao chép tên kệ như hiển thị ở đây:

nhập mô tả hình ảnh ở đây


3) Điều hướng đến khung "Đang chờ thay đổi", nhấp vào "Kệ" và dán tên kệ ví dụ:

nhập mô tả hình ảnh ở đây


4) Nhấn nút Có trên hộp thoại xác minh thay thế giá:

nhập mô tả hình ảnh ở đây


7
Vấn đề với giải pháp này là nó mang tính hủy diệt trong đó tất cả các ý kiến ​​đều bị thổi bay. Tôi nghĩ rằng một giải pháp tốt hơn sẽ là tạo ra một đánh giá mã mới và đặt cái cũ làm một mục liên quan.
Bardia

15
Thay thế các kệ không chạm vào các bình luận chỉ các tập tin. Nhóm của chúng tôi sử dụng phương pháp này thành công và chúng tôi không mất bất kỳ nhận xét đánh giá nào.
Chad Dienhart

10
Điều này hoạt động, nhưng nếu bạn thêm / xóa các dòng khi bạn chỉnh sửa tệp và thay thế giá, các nhận xét trên tệp đó sẽ trỏ đến các dòng sai.
Cằm

2
Tôi thường không muốn lặp lại, nhưng quan điểm của @ Chin ở trên khiến câu trả lời này hoàn toàn không thể thực hiện được. Có, bạn có thể dễ dàng cập nhật giá sách của mình nhưng làm như vậy về cơ bản TẤT CẢ các nhận xét đánh giá trước đây hiện đang trỏ đến văn bản SAU. (Vì rõ ràng một nhận xét đánh giá được đính kèm với phần bù tuyệt đối từ đầu tệp.)
Michael Sorens

7
Quá thủ công và không phải là điều lý tưởng. TFS hoặc Visual Studio nên có một cái gì đó để so sánh các thay đổi giữa các đánh giá liên tiếp, để người đánh giá có thể thấy chính xác những thay đổi được thực hiện kể từ lần đánh giá cuối cùng. Đừng tồn tại bất cứ điều gì như thế này? Có thể là một plugin hoặc một cái gì đó ...
Iúri dos Anjos

6

Tôi tin rằng quy trình chính xác là trước Người 1 để thực hiện các thay đổi và yêu cầu xem xét lại. Khi mã của bạn cần hoạt động có nghĩa là bạn sẽ thay đổi nó, vì vậy bạn sẽ muốn có phiên bản cũ để xem lại để so sánh. Bạn vẫn có đánh giá cũ trong lịch sử sau khi nó bị đóng nếu bạn muốn xem qua các bình luận. Chúng tôi hiện đang trong quá trình tối ưu hóa quy trình xem xét mã tại nơi làm việc của tôi.


2

Tôi biết câu hỏi này đã cũ, nhưng nó vẫn không được hỗ trợ như được chỉ ra bởi các áp phích khác. Giải pháp được đề xuất bởi chad sẽ hoạt động cho một số thứ nhưng sẽ có hành vi kỳ quặc cho những người khác.

Gần đây, nhóm TFS đã bắt đầu các giai đoạn lập kế hoạch cho một giải pháp cho một yêu cầu sử dụng rất cũ để cho phép cập nhật Đánh giá mã để giải quyết vấn đề này một cách tao nhã bằng cách thực hiện Đánh giá mã.


1
Tuyệt quá. Tôi sẽ mong chờ nó quá. Tôi sẽ hoan nghênh một giải pháp thực sự. Bài viết của tôi chỉ đơn giản là một cách để giải quyết vấn đề.
Chad Dienhart

Hy vọng rằng nó sẽ không giống như các yêu cầu kéo DevOps của azure. Họ là một cơn ác mộng spam.
StingyJack

0

Bạn phải làm điều này với hai đánh giá khác nhau. Nhưng đó cũng là một cách để giữ lịch sử với đánh giá thứ hai. Tất cả bạn cần là nhiệm vụ.

Quy trình công việc này được mô tả cho các đánh giá dựa trên thay đổi, nhưng nó cũng hoạt động cho các đánh giá dựa trên giá trị.

  1. Tạo task1
  2. Trước khi kiểm tra thay đổi1, hãy thêm task1 làm mục công việc liên quan
  3. Kiểm tra các thay đổi với một mục công việc liên quan và yêu cầu xem lại bộ thay đổi này
  4. Tạo task2
  5. Trước khi kiểm tra thay đổi2, thêm cả hai tác vụ như một mục công việc liên quan.
  6. Kiểm tra các thay đổi với hai mục công việc liên quan và yêu cầu xem lại bộ thay đổi này

Bây giờ trong yêu cầu đánh giá thứ hai, người đánh giá có thể tìm kiếm các nhiệm vụ liên quan và nếu người đánh giá xem xét nhiệm vụ1, anh ta / cô ta sẽ thấy các thay đổi1 và yêu cầu đánh giá với các nhận xét. Vì vậy, bạn sẽ không bị mất lịch sử cuộc trò chuyện.

Khi sử dụng trang web của chúng tôi, bạn xác nhận rằng bạn đã đọc và hiểu Chính sách cookieChính sách bảo mật của chúng tôi.
Licensed under cc by-sa 3.0 with attribution required.