2013-08-19 45 views
88

Chúng tôi đang bắt đầu sử dụng chức năng đánh giá mã được tích hợp sẵn cho bản xem trước VS 2012 và VS 2013. Yêu cầu xem xét và thêm nhận xét có vẻ khá đơn giản. Nếu ai đó thêm nhận xét yêu cầu mã được thay đổi, 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?Xem lại mã TFS - Hiển thị các tập tin cập nhật theo các ý kiến ​​

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

  1. Person 1 yêu cầu xem xét lại mã.
  2. Người 2 thêm nhận xét và chọn "Cần làm việc".
  3. Người 1 thực hiện các thay đổi cần thiết.

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

Tạo bài đá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 bạn thực hiện thay đổi.

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

+0

+1 cho "có trang web nào hiển thị các phương pháp hay nhất" không. 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 nhìn thấy một số suy nghĩ và kịch bản để sử dụng Code Review. – DaveShaw

Trả lời

112

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

  1. Person 1 yêu cầu xem xét lại mã.
  2. Người 2 thêm nhận xét và chọn "Cần làm việc".
  3. Người 1 thực hiện các thay đổi cần thiết.
  4. Person 1 cập nhật các shelveset liên quan đến việc xem xét mã
  5. Person 1 cho biết 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 chấp nhận

Dưới đây là các bước cần thiết để cập nhật các shelveset liên quan với việc xem xét.

  1. Từ "Mã đánh giá" cửa sổ chọn "xem shelveset" liên kết
  2. Từ "Shelveset Details" cửa sổ nổi bật và sao chép tên shelveset
  3. Điều hướng đến "Pending Changes" cửa sổ, bấm vào "xiên" và dán tên shelveset
  4. Nhấn nút Yes trên shelveset thay thế hộp thoại xác minh
  5. Bây giờ người xem có thể xem các tập tin cập nhật và các cuộc thảo luận xem xét có thể tiếp tục

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


1) Từ "Mã đánh giá" cửa sổ chọn "xem shelveset" liên kết như ở đây:

enter image description here


2) Từ "Shelveset Details" cửa sổ nổi bật và sao chép tên kệ sách như được hiển thị ở đây:

enter image description here


3) Điều hướng đến "Pending Changes" cửa sổ, bấm vào "xiên" và dán tên shelveset ví dụ:

enter image description here


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

enter image description here

+6

Vấn đề với giải pháp này là nó phá hoại ở chỗ tất cả các bình luận đều bị thổi bay đi.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à thiết lập một cái cũ như một mục liên quan. – Bardia

+11

Thay thế ngăn kệ không chỉ chạm vào các bình luận các tệp. 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 xóa bất kỳ nhận xét đánh giá nào. –

+0

Đây là câu trả lời hay nhất cho đến nay. Tôi sẽ chia sẻ điều này với nhóm của chúng tôi. – John

5

Tôi tin rằng thủ tục chính xác là fore Người 1 thực hiện thay đổi và yêu cầu đánh giá khác. Khi mã của bạn cần công việc có nghĩa là bạn sẽ thay đổi nó, do đó 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 bị đóng nếu bạn muốn xem qua các nhận xét. Chúng tôi hiện đang trong quá trình tối ưu hóa quy trình xem xét mã của chúng tôi tại nơi làm việc của tôi.

0

Bạn phải thực hiện việc này với hai bài đá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ả những gì bạn cần là nhiệm vụ.

Luồng công việc này được mô tả cho các bài đá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á.

  1. Tạo task1
  2. Trước khi check-in changeset1 thêm task1 như một hạng mục công trình có liên quan
  3. Kiểm tra những thay đổi với mục một công việc liên quan và yêu cầu xem xét lại để changeset này
  4. Tạo task2
  5. Trước khi check-in changeset2 thêm cả hai nhiệm vụ như một mục công việc liên quan.
  6. Kiểm tra những thay đổi với mục hai công việc liên quan và yêu cầu xem xét lại để changeset này

Bây giờ trong yêu cầu xem xét thứ hai người xem có thể tìm kiếm các công việc có liên quan và nếu người xem có một cái nhìn cho task1 anh/cô ấy xem changeset1 và yêu cầu xem xét với các nhận xét của nó. Vì vậy, bạn sẽ không bị mất lịch sử trò chuyện.

1

tôi biết câu hỏi này là cũ, nhưng nó vẫn không được hỗ trợ như đã chỉ ra bởi ot áp phích của cô ấy. Các giải pháp được đề xuất bởi chad sẽ làm việc cho một số điều nhưng sẽ có hành vi kỳ quặc cho người khác.

Gần đây, nhóm TFS đã bắt đầu các giai đoạn lập kế hoạch cho giải pháp cho a very old uservoice request to enable updating of a Code Review cần giải quyết vấn đề này một cách thanh lịch bằng cách làm cho Code Reviews có lặp lại.

+0

Tuyệt vời.Tôi sẽ được mong chờ nó quá.Tôi sẽ chào đón 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 để làm việc xung quanh vấn đề. –