Bàn thắng
Quy trình xem xét của chúng tôi có một số mục tiêu:
- Đảm bảo mã chất lượng cao, về cả chức năng và khả năng đọc.
- Phát hiện lỗi, bởi vì có thể xảy ra lỗi.
- Duy trì kiểu nhất quán để dễ dàng bắt đầu làm việc trong bất kỳ phần nào của cơ sở mã.
Toàn bộ mã đưa vào blockly-samples và core Blockly đều được xem xét, cho dù là do người đóng góp trong cộng đồng hay thành viên trong nhóm Blockly viết.
Với tư cách là người đánh giá, chúng tôi cố gắng phối hợp với bạn để làm cho thay đổi của bạn tốt nhất có thể. Với tư cách là cộng tác viên, chúng tôi đề nghị bạn trò chuyện với chúng tôi để yêu cầu lấy dữ liệu của bạn sau khi được xem xét và hợp nhất.
Quá trình
Quy trình đánh giá PR sẽ trải qua một số giai đoạn:
Assignment
Khi yêu cầu lấy dữ liệu của bạn được gửi đến, thành viên trực điện thoại của nhóm Blockly sẽ chỉ định một người đánh giá.
Người đánh giá được lựa chọn dựa trên chuyên môn và để phân bổ đồng đều khối lượng công việc.
Người đánh giá có thể mất vài ngày để được chỉ định và vài ngày nữa để được xem xét. Đừng lo lắng, đây là điều bình thường.
Ý kiến phản hồi
Trong giai đoạn phản hồi, người đánh giá sẽ đưa ra đề xuất về những thay đổi đối với PR của bạn. Đây có thể là những yếu tố đơn giản để giúp mã của bạn tuân theo Hướng dẫn về quy tắc lập trình JavaScript của Google. Hoặc chúng có thể là những nội dung lớn hơn như yêu cầu bạn sắp xếp lại các định nghĩa hàm.
Người đánh giá nên sử dụng quy trình xem xét mã của GitHub (thay vì nhận xét riêng lẻ) để bạn chỉ nhận được một thông báo thay vì nhiều thông báo.
Thảo luận
Giai đoạn thảo luận là cơ hội để bạn trả lời ý kiến phản hồi. Có thể một trong các nhận xét đánh giá không rõ ràng: bây giờ là cơ hội để bạn yêu cầu làm rõ. Hoặc có thể người đánh giá đã yêu cầu thay đổi, nhưng bạn cho rằng thay đổi đó sẽ có hậu quả: giờ là cơ hội để bạn tìm ra giải pháp.
Bản sửa đổi
Giai đoạn sửa đổi là nơi bạn điều chỉnh lời PR của mình. Thường thì những thay đổi này là kết quả của điều người đánh giá đã đưa ra trong giai đoạn phản hồi.
Sau khi hoàn tất các bản sửa đổi, bạn nên gắn thẻ cho người đánh giá để mời họ xem lại.
Lặp lại
Sau giai đoạn sửa đổi, người đánh giá có thêm một cơ hội để phản hồi và quy trình sửa đổi sẽ bắt đầu lại từ đầu.
Thông thường, quá trình xem xét thứ hai rất đơn giản và tập trung vào các nội dung nit (chẳng hạn như dấu câu và kiểu mã). Nhưng đôi khi việc xem xét thứ hai có thể khá lớn. Người đánh giá đầu tiên của bạn thậm chí có thể yêu cầu người khác xem xét để có được góc nhìn mới.
Hợp nhất!
Giai đoạn hợp nhất chính là cơ hội để bạn kỷ niệm. Bạn đã tạo một thay đổi, thảo luận, sửa đổi và cuối cùng đã hợp nhất thay đổi đó! Đây là một thành tích lớn mà nhiều người không bao giờ bắt đầu, chứ chưa nói đến việc hoàn thành!
Cảm ơn bạn đã nỗ lực hết mình để cải thiện Blockly. Xin chúc mừng!