Làm cách nào để xem xét mã bằng TFS?

Question

thể trùng lặp:
Web Based Code-Review tools for Team Foundation Server

Chúng tôi có khoảng 4 nhà phát triển và chúng tôi đang sử dụng TFS để kiểm soát nguồn.

Chúng tôi muốn giới thiệu Code Reviewing để mã được đăng ký được các thành viên ngang hàng xem xét về hiệu suất, thiết kế và tiêu chuẩn.

Bạn muốn sử dụng công cụ nào để giúp điều này? (nguồn mở và hy vọng miễn phí)

Tôi tìm thấy điều này từ Google: http://teamreview.codeplex.com/ và tôi đang cố gắng kiểm tra cách hoạt động.

Cảm ơn,

Answer 1

Tôi không biết về bất kỳ công cụ, cho mỗi gia nhập, nhưng khi sử dụng TFS trong một nhóm nhỏ, tôi thấy rằng chỉ đơn giản là sao lưu những thay đổi trên một shelveset là đủ. Bằng cách này, các nhà phát triển khác có thể xem các kệ sách của bất kỳ ai khác và bước qua những khác biệt về mã trong thời gian riêng của họ. Họ cũng có thể unshelve những thay đổi trên máy riêng của họ nếu họ muốn kiểm tra việc biên dịch và thực hiện các mã.

Điều gì đó khác mà bạn có thể thực hiện để hỗ trợ trong việc đảm bảo mã đúng, viết các bài kiểm tra đơn vị và đặt chúng dưới dạng xem xét mã. Bằng cách đó, hệ thống kiểm tra có thể giúp bạn xác định các khuyết tật về hành vi tiềm năng có thể bị mất bởi con mắt của một người đánh giá mã.

Answer 2

Tôi không chắc chắn điều này sẽ áp dụng cho bạn như thế nào, nhưng ở đây sẽ xảy ra. Tôi đã thay đổi mẫu quy trình WIT của chúng tôi (sử dụng TFS power tools) để trạng thái sau khi đăng ký mã là "Đang chờ xem xét mã". Sau đó, bạn cần phải gán WI này cho người dùng khác, những người di chuyển nó vào trạng thái "Closed" nếu xem xét mã là OK, hoặc quay trở lại để gán nếu nó không thành công.

Vì vậy, các hiệu ứng chuyển tiếp quá trình hiện nay có cái gì đó như:

 
Assigned --Check-In-->Code Review --Success-->Closed 
    ^-------Failed---------| 

này còn xa mới hoàn hảo, và nó chủ yếu dựa vào các quá trình chúng tôi đã có với Subversion và một hệ thống theo dõi lỗi riêng biệt, nhưng nó hoạt động.

Trong các tin tức khác, VS 11 có vẻ như là out of the box (cũng là here - được lấy từ câu trả lời của Ewald).

Answer 3

Nó sẽ không giúp bạn ngay bây giờ, nhưng Dev11 (hiện có sẵn dưới dạng Bản xem trước dành cho nhà phát triển) đã bao gồm đánh giá mã. Bạn có thể xem tổng quan tốt đẹp tại session from Jamie Cool at //BUILD.

Vì vậy, trước khi đưa ra quyết định lớn, bạn có thể muốn đợi một vài tháng trước khi bạn có thể sử dụng công việc này hàng ngày.

Ewald Hofman - Người quản lý chương trình TFS

Answer 4

Tôi có thể đề xuất Review Assistant code review tool. Nó tích hợp vào Visual Studio 2010 và 2012, và làm việc với TFS. Như tôi thấy từ bài đăng của bạn, quy trình xem xét mã sau cam kết là thích hợp hơn cho bạn. Mô hình chính xác này được thực hiện trong công cụ.