AllianceProject Handbook
← Thuật ngữ

Mã nguồn và review

pull request

còn gọi là yêu cầu gộp mã

Đề nghị gộp một nhánh vào nhánh chính, kèm chỗ để người khác đọc và bình luận trước khi gộp.

Pull request là chỗ dừng bắt buộc giữa "tôi viết xong" và "mã vào nhánh chính". Nó gom ba thứ vào một chỗ: phần thay đổi, kết quả CI, và ý kiến người review.

PR nhỏ được review kỹ; PR lớn được review qua loa. Đây là quy luật thực nghiệm, không phải lời khuyên đạo đức — quá vài trăm dòng thì người đọc chuyển từ tìm lỗi sang lướt cho xong.

Mô tả PR viết cho người review, không viết cho máy: thay đổi gì, vì sao, đã tự kiểm thế nào.

Bài dùng thuật ngữ này