Trên GitHub, sự khác biệt giữa người đánh giá và người được chỉ định là gì?

Question

Một tính năng bổ sung trên Đéc, 7, năm 2016, công bố trên GitHub blog, giới thiệu tùy chọn to add reviewers to a Pull Request

Bây giờ bạn có thể yêu cầu xem xét một cách rõ ràng từ cộng tác viên, làm cho nó dễ dàng hơn để xác định những người bạn' d muốn xem xét yêu cầu kéo của bạn.

Bạn cũng có thể xem danh sách những người bạn đang chờ xem xét từ thanh bên trang yêu cầu kéo, cũng như trạng thái đánh giá từ những người đã rời khỏi họ.

Tuy nhiên, thiết lập rõ ràng người đánh giá cho PR đã được thực hiện bằng cách chỉ định người (người được giao tùy chọn).

Với cả hai tùy chọn hiện có sẵn, vai trò của từng tùy chọn là gì vì cả hai đều có cùng mục tiêu cuối cùng?

Answer 1

EDIT:

Sau khi bàn bạc với một số bảo trì OSS, xét được định nghĩa là những gì từ nghĩa vụ phải được: để xem xét (mã của một ai đó) và "chuyển nhượng" đã một definiton lỏng hơn giải thích dưới đây .

Đối với "người đánh giá": người mà bạn muốn xem lại mã. Không nhất thiết phải là người chịu trách nhiệm về khu vực đó hoặc chịu trách nhiệm sáp nhập cam kết. Có thể là một người làm việc trên đoạn mã đó trước đây, như GitHub tự động gợi ý.

Đối với "người được chuyển nhượng": tối đa nhóm dự án/người duy trì ý nghĩa của nó và không có định nghĩa nghiêm ngặt. Nó có thể là người mở PR, hoặc ai đó chịu trách nhiệm cho khu vực đó (người sẽ chấp nhận PR sau khi xem xét xong hoặc chỉ đóng nó). Nó không phải để GitHub để xác định những gì nó được để nó mở cho các nhà bảo trì dự án những gì phù hợp nhất cho dự án của họ.

câu trả lời trước:

Ok tôi sẽ đi trước và trả lời câu hỏi của riêng tôi.

Đối với PR của người dùng có quyền truy cập ghi: Người được chỉ định sẽ là người mở PR và người đánh giá sẽ thay thế chức năng người được chỉ định cũ (xem mã), đây là người được người được chỉ định lựa chọn.

Đối với PR của người dùng không có quyền truy cập ghi (bên ngoài cộng tác viên): Người nào đó có quyền ghi sẽ giao cho mình (hoặc thành viên viết riêng), xem xét PR (Người phản biện). Người được chỉ định trống.

Đối với PR chưa hoàn thành từ các cộng tác viên bên ngoài: thành viên truy cập sẽ thực hiện công việc chưa hoàn thành và chỉ định cho cô ấy. Cô sẽ chịu trách nhiệm hoàn thành nhiệm vụ, là Người được giao. Vì lý do chính của PR là xem xét các thay đổi, cô ấy sẽ chọn một số người khác để xem xét các thay đổi.

Answer 2

Trong GitHub người đánh giá là người đánh giá yêu cầu kéo. Một chủ dự án có thể yêu cầu xem xét từ bất kỳ người bảo trì nào, Họ thậm chí có thể thiết lập một tùy chọn để yêu cầu kéo chỉ có thể được hợp nhất nếu nó được xem xét bởi một người duy trì có quyền ghi.

Theo chính thức github documentation, Người được chuyển nhượng là người đang làm việc về các vấn đề cụ thể và yêu cầu kéo. Đôi khi nó bị nhầm lẫn với tư cách là người đánh giá. Nó thực sự có nghĩa là để được sử dụng với các vấn đề hơn là kéo yêu cầu để khi chúng tôi nhận được một vấn đề chúng tôi có thể gán cho ai đó để sửa chữa nó. Trong yêu cầu kéo, người được chuyển nhượng đề cập đến người chịu trách nhiệm sáp nhập yêu cầu kéo sau khi nhận được nhận xét và yêu cầu thay đổi từ những người bảo trì khác.

Answer 3

Theo câu trả lời được chấp nhận. Có, "người được chuyển nhượng" có định nghĩa lỏng hơn và có thể được sử dụng khác nhau để phù hợp với nhu cầu của một nhóm.

Trong nhóm 8 nhà phát triển của chúng tôi, trong hầu hết các PR, chúng tôi có 1 người đánh giá, người đề xuất thay đổi và cuối cùng phê duyệt PR. Trong giai đoạn xem xét, "người được chuyển nhượng" là người đã mở PR; sau này nếu PR được chọn bởi nhà phát triển khác, một "người được chuyển nhượng" mới được thêm vào. Khi PR được chấp thuận và sẵn sàng cho QA hoặc hợp nhất trực tiếp, một "người được chỉ định" mới được thêm vào. Bằng cách này, danh sách "người được chuyển nhượng" phát triển.

Chúng tôi sử dụng "chuyển nhượng" để chỉ định những người sau đây gọi chung là:

1. Kéo Yêu cầu Tác giả
2. Author làm việc trên gợi ý thay đổi PR (thường giống như 1)
3. QA người tham gia
4. Người chịu trách nhiệm hợp nhất (thường giống với 2 hoặc 3)

Sử dụng "người được chuyển nhượng" giúp định vị PR trong tương lai eas ily. Một trong những dự án của tôi có> 3000 PRs.

is:open is:pr author:raya-dumas

is:closed is:pr assignee:raya-dumas

và truy vấn tương tự khác để giảm bớt quá trình tìm kiếm. "các cột mốc" là khá hữu ích để sử dụng cũng như để giảm bớt tìm kiếm PR.