2013-10-09 15 views
7

Tôi đang cố định cấu hình quy trình xem xét mã cho máy chủ TFS 2012 mới của nhóm của tôi. Hiện nay có một số điều mà chúng ta không thể tìm ra cách để làm bằng cách sử dụng chức năng xem xét mã tích hợp trong TFS/VS 2012:Giải pháp thay thế mã cho TFS 2012?

  1. Yêu cầu đã được phê duyệt/mã số nhận xét đã quyết toán cho bất kỳ mã checked-in. Cụ thể hơn, hãy thông báo cho nhà phát triển bất cứ khi nào họ đã đăng ký mã mà không có đánh giá mã được liên kết cho đến khi họ giải quyết.
  2. Cho phép nhà phát triển yêu cầu xem xét mã, cho phép người đánh giá đưa ra nhận xét và đánh dấu bài đánh giá là "cần làm việc", sau đó cho phép nhà phát triển thực hiện thay đổi mã và liên kết chúng với đánh giá mã ban đầu, đưa ra nhận xét gửi lại cho người đánh giá và quá trình này lặp lại vô thời hạn cho đến khi người đánh giá chấp thuận việc xem xét mã. Chúng tôi KHÔNG muốn thực hiện các đánh giá mã mới và liên kết chúng với các đánh giá mã trước đó như là công việc của TFS cho thấy.

Theo như tôi biết, bộ giải mã cho quá trình xem xét mã tích hợp TFS 2012 là dấu đầu dòng, tôi không thể tìm thấy giải pháp cho vấn đề này và có vẻ như không có cách nào xung quanh nó. Vì vậy, những gì tôi đang tìm kiếm là một giải pháp thay thế mã của bên thứ ba có thể tích hợp vào điều khiển nguồn TFS của chúng tôi hoặc một cách để thay đổi TFS để đáp ứng các yêu cầu của chúng tôi.

+0

Đối với # 2 tôi đã đăng một cách để thực hiện việc này tại đây: http://stackoverflow.com/questions/18322791/tfs-code-reviews-show -updated-files-in-response-to-comments/21126092 # 21126092 Rất tiếc, yêu cầu này chỉ yêu cầu thêm một vài bước. –

Trả lời

4

Bạn đúng, đây là hai khoảng trống phổ biến mà mọi người gặp phải khi thử chức năng Đánh giá mã TFS.

Đối # 1, thực thi một chính sách Mã Rà soát, nó không được hỗ trợ ra khỏi hộp nhưng Colin Dembovsky đã tạo ra một tùy chỉnh tùy theo chính sách có thể thực hiện điều này: http://visualstudiogallery.msdn.microsoft.com/c476b708-77a8-4065-b9d0-919ab688f078

Đối # 2, TFS hiện không hỗ trợ các đánh giá mã "đa lần lặp". Cách duy nhất để xử lý điều này trong TFS hôm nay là khởi động một Code Review mới.

Dev Requests Code Review 
Reviewer provides comments 
Dev Closes Code Review 
Dev fixes code 
Dev Creates new Code Review 
... 
+2

Cảm ơn bạn đã cung cấp thông tin, tiếc là # 2 là một giao dịch cho chúng tôi. –

3

Tôi có thể đề xuất Review Assistant làm phương án thay thế cho chức năng Đánh giá mã TFS.

Công cụ này thực hiện đầy đủ số 2 của bạn - đánh giá mã "nhiều lần lặp".

Điểm # 1 (chính sách đăng ký tùy chỉnh) chưa được triển khai nhưng trên lộ trình của chúng tôi.

Đánh giá Trợ lý nếu được tích hợp vào Visual Studio cũng như tính năng tiêu chuẩn. Nhưng bạn có thể sử dụng nó với phiên bản chuyên nghiệp của VS, và không chỉ trong Premium hoặc Ultimate.

+0

Chúng tôi đã cài đặt và đánh giá Trợ lý đánh giá không may là nó không hỗ trợ chính sách đánh giá mã đăng ký, chúng tôi cần phải có khả năng thực thi mọi kiểm tra được bao gồm trong Xem xét mã nếu không tác giả của changeset sẽ được cảnh báo và chúng tôi cần có thể xem báo cáo tổng thể về bất kỳ mã chưa được phát hiện nào.Vấn đề khác là chúng tôi thường bao gồm các bài kiểm tra đơn vị trong các đánh giá mã và không có cách nào để thêm một trường văn bản tùy chỉnh vào một đánh giá mã để hỗ trợ điều này. –

+0

Như tôi đã đề cập, hiện tại chúng tôi không hỗ trợ chính sách đánh giá, nhưng chúng tôi dự định. Nhưng vấn đề với các bài kiểm tra đơn vị không rõ ràng. Chúng tôi sẽ đánh giá cao một số giải thích thêm. – Devart

+0

Chúng tôi yêu cầu bất kỳ thay đổi mã nào được bao gồm bởi việc xem xét mã và mọi đánh giá mã đều được liên kết với một bài kiểm tra đơn vị. Trong trường hợp của chúng tôi nó sẽ đủ để có thể liên kết một đánh giá mã cho một đơn vị kiểm tra/Test Case Work Item trong TFS. –

0

Trong TFS, các đánh giá mã trước được thêm dưới dạng mục công việc được liên kết vào đánh giá mã hiện tại. Tôi khuyến khích sử dụng quy trình đăng ký để đóng đánh giá mã cuối cùng được liệt kê là "Có vẻ tốt". Điều này sẽ đánh dấu việc đóng mã xem xét là mã được đăng ký.

Các vấn đề liên quan