Làm cách nào để thiết lập đánh giá mã bằng Gitlab?

Làm cách nào để thiết lập một đánh giá mã bằng Gitlab? Tôi thấy nó được liệt kê dưới dạng một tính năng trên trang web Gitlab, nhưng dường như tôi không thể tìm thấy hướng dẫn về cách thiết lập một tính năng này (Đối với vấn đề đó, bất kỳ liên kết nào đến hướng dẫn sử dụng Gitlab sẽ được đánh giá cao nhất).

Một số tìm kiếm của tôi đã chỉ ra rằng 'Yêu cầu hợp nhất' là cách để thực hiện ... nhưng tôi thấy chúng hạn chế. Một yêu cầu hợp nhất được ban hành cho thấy tất cả các cam kết giữa một nhánh và nhánh kia. Tôi dường như chỉ có thể xem các khác biệt được tạo cho mỗi cam kết riêng lẻ. Ví dụ: giả sử tôi có một tệp muốn xem lại. Đó là một tệp mới nhưng tôi đã gửi các thay đổi cho nó hơn 10 lần cam kết trên một nhánh nhà phát triển. Nếu tôi đưa ra một yêu cầu hợp nhất cho nhánh nhà phát triển đó từ quá trình tích hợp, tôi thấy 10 cam kết, mỗi cam kết hiển thị những thay đổi gia tăng được thực hiện đối với tệp ... Tôi muốn xem lại toàn bộ. Cai nay moi!

Có phải tôi đang sủa nhầm cây ở đây không? Có công cụ xem xét mã thực tế nào mà tôi có thể sử dụng trong GitLab không, hay các yêu cầu hợp nhất đang diễn ra và nếu có thì tôi đang sử dụng chúng không đúng cách? cách tốt nhất để thiết lập một đánh giá mã thích hợp ở đây là gì?

Lưu ý: vì GitLab 6.4, có sẵn dạng xem khác biệt song song : xem " yêu cầu kéo 5308 ".

(Tháng 7 năm 2013)Tuy nhiên, vẫn chưa có khả năng bình luận về từng dòng, chỉ ở cấp tệp.
Daniel Sokolowski đề cập trong phần nhận xét rằng Nhận xét trên mỗi dòng hiện được hỗ trợ (09/2014):

Các thành viên trong nhóm của bạn có thể nhận xét về yêu cầu hợp nhất nói chung hoặc trên các dòng cụ thể với nhận xét dòng.

Điều đó vẫn có thể giúp ích cho hoạt động xem xét mã.

6 năm sau, đối với GitLab 13.1 (tháng 6 năm 2020) :

Yêu cầu hợp nhất Đánh giá được chuyển đến Core

Ban đầu được giới thiệu trong GitLab 11.4 như một tính năng GitLab Premium, Đánh giá yêu cầu hợp nhất cho phép người đánh giá yêu cầu hợp nhất:

• gửi nhiều bình luận cùng một lúc,
• giảm nhiễu thông báo cho tác giả yêu cầu hợp nhất và
• cho phép quy trình xem xét hợp lý và chặt chẽ hơn.

Kể từ khi được giới thiệu, chúng tôi đã đánh giá lại vị trí của nó trong mô hình định giá dựa trên người mua và là một phần của 13.1, chúng tôi vui mừng thông báo rằng tính năng này hiện đã được chuyển sang GitLab Core.

Xem Tài liệu và Issue

Tôi đã thực hiện đánh giá mã trong Gitlab trong hơn hai tháng mà hầu như không gặp bất cứ khó khăn nào. Tôi đã thiết lập rss2email để gửi thông báo qua email mỗi khi nhà phát triển đưa ra cam kết mới. Sau đó, tôi sử dụng tính năng bình luận của Gitlab để cam kết thực hiện một số nhận xét về mã được đẩy.

Thật không may, Gitlab không cho phép nhận xét trên chính tệp, chỉ trong cam kết (giống như Github, tôi đoán vậy). Bất cứ khi nào tôi thấy mình trong một tình huống mà tôi cần nhận xét điều gì đó mà tôi đã bỏ lỡ trong một lần cam kết trước đó, tôi sử dụng công cụ đổ lỗi để tìm cam kết đã giới thiệu / thay đổi phần mã được nhận xét.

Nó còn lâu mới hoàn hảo, nhưng nó đang hoạt động tốt cho đến nay.

Bạn có thể thấy mã đã gửi trong Yêu cầu hợp nhất cho kho lưu trữ khác hoặc trong kho lưu trữ hiện tại.
ví dụ http://demo.gitlab.com/diaspora/diaspora/commits/master

Sau đó, bạn có thể thêm nhận xét vào các thay đổi tệp đã cam kết (nút Reply) hoặc toàn bộ cam kết

ví dụ http://demo.gitlab.com/diaspora/diaspora/commit/42f47626890218a180870bc3f44ec57625b0779c

Kết quả giao tiếp là xem xét mã . Tuy nhiên, cá nhân tôi khuyên bạn nên thực hiện đánh giá mã trên một PC với giao tiếp trực tiếp bất cứ khi nào có thể và sử dụng các công cụ để ghi lại kết quả hoặc khi cần hình thức hơn.

Đối với bản sửa đổi tệp có nhiều cam kết, ví dụ: http://demo.gitlab.com/diaspora/diaspora/blame/master/README.md, hãy xem nó bằng cách sử dụng blameđể hiểu ai đã làm gì. Tuy nhiên trong quan điểm này không có tùy chọn để giao tiếp và thêm nhận xét. Tôi khuyên bạn chỉ nên thêm các thay đổi dưới dạng nhận xét trong trường hợp này.

Đúng. Yêu cầu hợp nhất là cách thực hiện đánh giá ngang hàng.

Nên có một tab 'khác biệt' sẽ hiển thị các thay đổi của tất cả các cam kết (được đề cập ở đây: http://youtu.be/DyAX8ws5OIc?t=3m2s ).

Video cũng giải thích độc đáo cách nó có thể được sử dụng để đánh giá ngang hàng.

Trường hợp sử dụng thông thường của việc xem xét mã là xem xét mã trên một nhánh trước khi hợp nhất thành chính hoặc tương tự. Tôi có một tình huống là tôi đã phát triển một dự án và muốn tất cả mã được mọi người trong nhóm xem xét.

Những gì tôi đã làm là:

Kiểm tra cam kết đầu tiên, thực hiện thay đổi đối với nó, cam kết và đẩy

git co -b FIRST_COMMIT eb67f06c2b3222c0219214b176c41922bc454881
vi README.md
git add README.md
git ci -m "First commit modified so can get full diff against it"
git push --set-upstream origin FIRST-COMMIT

Kiểm tra cam kết cuối cùng, thực hiện thay đổi đối với nó, cam kết và đẩy

git co -b master
vi README.md
git add README.md
git ci -m "Last commit modified so can get full diff against it"
git push --set-upstream origin LAST-COMMIT

Trên GitLab / GitHub, hãy tạo một yêu cầu kéo

• Nó là một hợp nhất từ ​​LAST_COMMIT thành FIRST_COMMIT

Làm việc cho tôi!