Tài liệu này hướng dẫn cách xem xét một thay đổi. Hãy lưu ý các tiêu chí sau đây đối với một thay đổi hợp lệ:
- Thay đổi này có phù hợp với mục đích đã nêu của dự án này không?
- Thay đổi này có hợp lệ trong cấu trúc hiện có của dự án không?
- Thay đổi này có gây ra lỗi thiết kế có thể gây ra vấn đề trong tương lai không?
- Thay đổi này có tuân thủ các phương pháp hay nhất đã được thiết lập cho dự án này không?
- Thay đổi này có phải là một cách hay để thực hiện chức năng được mô tả không?
- Thay đổi này có gây ra bất kỳ rủi ro nào về bảo mật hoặc tính không ổn định, hoặc làm hỏng bất kỳ ứng dụng hiện có nào không?
Để biết thêm thông tin về việc xem xét một thay đổi, hãy xem phần Xem xét một thay đổi trong tài liệu Gerrit.
Nếu được chỉ định là người xem xét một thay đổi, bạn sẽ nhận được một email có đường liên kết đến thay đổi đó. Hãy làm theo các bước sau để xem xét và xem xét thay đổi:
Nhấp vào đường liên kết đến thay đổi trong email. Màn hình Thông tin thay đổi của Gerrit sẽ xuất hiện. Màn hình này cung cấp nội dung mô tả về thay đổi, xác định người đóng góp thay đổi và tất cả người xem xét, đồng thời cung cấp danh sách các tệp được liên kết với thay đổi đó.
Đọc nội dung mô tả thay đổi để hiểu mục tiêu của thay đổi đó.
Nhấp vào một tệp để xem xét các thay đổi trong tệp đó. Tệp sẽ mở ra trong màn hình so sánh hai ngăn, với tệp hiện có ở bên trái và tệp có các thay đổi được đề xuất ở bên phải. Cách xem xét tệp:
Dùng chuột để làm nổi bật (nhấp + kéo) phần mã mà bạn có nhận xét hoặc câu hỏi. Thông báo "Nhấn c để nhận xét" sẽ xuất hiện.
Nhấn c. Hộp nhận xét bản nháp sẽ xuất hiện.
Nhập nhận xét hoặc câu hỏi rồi nhấp vào LƯU.
Nhấp vào số duy nhất của thay đổi (phía trên bên trái màn hình) để quay lại màn hình Thông tin thay đổi.
Tiếp tục xem xét từng tệp trong thay đổi.
Khi bạn xem xét xong tất cả các tệp trong thay đổi, hãy quay lại màn hình Thông tin thay đổi rồi nhấp vào TRẢ LỜI. Màn hình Gửi phiếu bầu theo yêu cầu sẽ xuất hiện.
Chọn một trong các điểm số sau đây cho thay đổi:
- +1 Tôi thấy ổn, nhưng người khác cũng phải phê duyệt.
0 Không có điểm
-1 Tôi không muốn gửi thay đổi này ở trạng thái hiện tại, vì có những việc cần khắc phục.
-2 Không nên gửi thay đổi này
Nhấp vào TRẢ LỜI để gửi nhận xét và điểm số của bạn cho nhà phát triển thay đổi.
Nếu có vấn đề cần khắc phục, nhà phát triển sẽ khắc phục các vấn đề đó và yêu cầu bạn xem xét các bản sửa lỗi. Nếu không, bạn đã xem xét xong thay đổi.