Làm thế nào để thực hiện đánh giá ngang hàng về các yêu cầu kéo GitHub?


12

Chúng tôi đang chuyển từ Bitbucket sang GitHub và một điều chúng tôi đang đấu tranh là các đánh giá mã ngang hàng hoạt động rất trơn tru trên Bitbucket như thế này:

  1. Tác giả đã mở Yêu cầu kéo (GitHub: giống nhau)
  2. Tác giả đã thêm đồng nghiệp của mình làm người đánh giá (GitHub: ?? đấu tranh ở đây với nhiều người được giao)
  3. Người phản biện:
    1. Đã phê duyệt PR với dấu kiểm màu xanh lá cây (GitHub: ??)
    2. Đã thêm nhận xét (GitHub: giống nhau)
    3. Đã tạo các tác vụ nhẹ (GitHub: sắp xếp tương tự nếu - [ ]cú pháp được sử dụng trong mô tả PR; xấu hổ vì nó không hoạt động cho các tác vụ)
  4. Có một danh sách các PR mà tôi có thể thấy trong nháy mắt được xem xét và OK để được hợp nhất và cần được chú ý thêm (GitHub: ??)

Tôi nên chỉ ra rằng chúng tôi muốn tránh các công cụ đánh giá mã của bên thứ 3 nếu có thể và muốn ở lại trên vanilla GitHub với một số cách giải quyết.


1
Âm thanh như bạn có thể được chuyển đổi sớm. Tại sao phải chuyển đổi, đặc biệt là nếu điều mới không có tất cả các tính năng mà bạn cần?
mẫu

Viết bình luận cho prq của bạn và đánh dấu @ bất cứ khi nào bạn muốn nhận thông báo. Người đánh giá có thể thêm các thẻ để hiển thị ý kiến ​​đánh giá của họ.
Wilbert

Câu trả lời:


6

Từ những gì tôi đã thấy, hầu hết các bước đó đều được thực hiện trên Github theo quy ước chứ không phải bởi bất kỳ quy trình chính thức nào do Github cung cấp.

Chủ nhân của tôi sử dụng Github, tôi điều hành một số lượng lớn các dự án nguồn mở nhỏ và đóng góp không thường xuyên cho các dự án nguồn mở khác.

Đây là cách tôi thường thấy nó được thực hiện:

Tác giả thêm đồng nghiệp của mình làm người đánh giá:

Điều này thay đổi từ dự án này sang dự án khác, nhưng nói chung, những người đánh giá ngang hàng được chỉ định là tất cả những người đóng góp cho dự án .

Các dự án nguồn mở dường như có một hệ thống phân cấp thô - có thể quy ước của chúng sẽ chỉ hợp nhất sau khi một người đóng góp "cốt lõi" đã đồng ý.

Tại cửa hàng nơi tôi hiện đang làm việc, chúng tôi hợp nhất sau khi bất kỳ một trong số nửa tá nhà phát triển trong nhóm đã chấp thuận.

Trong những trường hợp hiếm hoi, ai đó trong nhóm có thể sử dụng một nhận xét để gọi cụ thể một nhà phát triển khác mà họ nghĩ nên xem lại mã trước khi hợp nhất, nhưng nếu không, bất cứ ai đến đó trước và cảm thấy muốn làm như vậy đều có thể xem xét và đưa ra nhận xét.

Phê duyệt phê duyệt:

Sự chấp thuận thường được thể hiện bằng cách đưa ra nhận xét về yêu cầu kéo có nội dung "+1" hoặc "lgtm" (có vẻ tốt với tôi).

Nhiệm vụ nhẹ:

Tôi cũng đã sử dụng các hộp kiểm, nhưng trong hầu hết các trường hợp, mọi nhận xét về yêu cầu kéo được coi là một "tác vụ" ngầm được giải quyết bằng cách:

  • thay đổi mã mà dòng đang bình luận
  • trả lời với một bình luận khác

Nhìn thoáng qua những gì được phê duyệt và những gì vẫn cần xem xét:

Tôi đã sử dụng tiện ích mở rộng Looks Good To Me cho Chrome, cung cấp cho bạn chế độ xem như vậy từ màn hình Yêu cầu kéo. Mặc dù vậy, chế độ xem danh sách yêu cầu kéo dường như đã bị phá vỡ bởi những thay đổi gần đây của Github.

Khi sử dụng trang web của chúng tôi, bạn xác nhận rằng bạn đã đọc và hiểu Chính sách cookieChính sách bảo mật của chúng tôi.
Licensed under cc by-sa 3.0 with attribution required.