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

Một tính năng được thêm vào ngày 7 tháng 12 năm 2016, được công bố trên blog GitHub, đã giới thiệu tùy chọn thêm người đánh giá vào Yêu cầu kéo

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

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

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

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

BIÊN TẬP:

Sau khi thảo luận với một số người duy trì OSS, người đánh giá được định nghĩa là từ được cho là: để xem lại (mã của ai đó) và "người được chuyển nhượng" có một định nghĩa lỏng lẻo được 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 là người chịu trách nhiệm cho khu vực đó hoặc chịu trách nhiệm sáp nhập cam kết. Có thể là người đã từng làm việc với đoạn mã đó trước đây, như GitHub tự động đề xuất.

Đối với "người được chuyển nhượng" : tùy theo nhóm / người bảo trì của dự án nghĩa là gì và không có định nghĩa chặt chẽ. 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 đánh giá được thực hiện hoặc chỉ đóng nó). GitHub không xác định những gì nó đang mở cho những người 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ẽ tiếp tụ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 ủy quyền sẽ là người đã mở PR và người đánh giá sẽ thay thế chức năng của người được chuyển nhượng cũ (xem lại mã), đây là một trong những lựa chọn của người được chuyển nhượng.

Đối với PR của người dùng không có quyền truy cập ghi (người đóng góp bên ngoài): Người nào đó có quyền truy cập ghi sẽ tự gán cho mình (hoặc thành viên viết thư khác), để xem lại PR (Người kiểm tra). Người nhận là trống.

Đối với PR chưa hoàn thành từ những người đóng góp bên ngoài : thành viên truy cập ghi sẽ nhận công việc còn dang dở và giao cho cô ấy. Cô sẽ chịu trách nhiệm hoàn thành nhiệm vụ, trở thành Người được ủy quyền . 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.

Trong GitHub, người đánh giá là người xem xét yêu cầu kéo. 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ể đặt tùy chọn sao cho yêu cầu kéo chỉ có thể được hợp nhất nếu được xem xét bởi một trong những người bảo trì có quyền truy cập ghi.

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

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 đội.

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 chấp thuận 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 nhà phát triển khác chọn, một "người được chuyển nhượng" mới sẽ được thêm vào. Khi PR được phê duyệt và sẵn sàng cho QA hoặc hợp nhất trực tiếp, một "người được chuyển nhượng" QA mới sẽ được thêm vào. 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 "người được chuyển nhượng" để chỉ định những người theo sau:

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

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

is:open is:pr author:raya-dumas

is:closed is:pr assignee:raya-dumas

Hoặc chỉ author:raya-dumasđể tìm tất cả các mục mà tác giả đã tạo (các vấn đề, PR)

và các 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" khá hữu ích để sử dụng cũng như để dễ dàng tìm kiếm PR.

Trước GitHub chỉ có trường người được chuyển nhượng và không có trường người đánh giá. Không có sự phân biệt trước đó vì vậy trường được chuyển nhượng thường được sử dụng làm trường đánh giá.

Nhưng sử dụng chúng bất cứ cách nào phù hợp với dự án của bạn.