Permintaan penarikan lebih mudah ditinjau jika tujuannya jelas dan perbedaannya menceritakan satu cerita yang koheren. Memisahkan pekerjaan yang tidak terkait mengurangi kemungkinan perubahan perilaku penting disembunyikan dalam pemformatan atau pembersihan.
Pilih batasan yang berguna
- Pisahkan pemfaktoran ulang mekanis dari perubahan perilaku agar hasilnya lebih mudah diverifikasi.
- Jaga agar pull request cukup kecil untuk ditinjau, namun jangan membaginya terlalu jauh sehingga masing-masing bagian tidak ada artinya.
- Jelaskan ketergantungan antara perubahan dan urutan perubahan tersebut.
Jadikan jalur peninjauan eksplisit
Jelaskan hasil pengguna atau sistem, pilihan implementasi yang penting, dan cara Anda mengujinya. Tautkan isu-isu terkait atau upaya tindak lanjut daripada secara diam-diam memperluas cakupan saat ini.
Sebelum meminta peninjauan, baca perbedaannya sebagai seseorang yang tidak terbiasa dengan tugas tersebut. Hapus kebisingan yang dihasilkan dan tandai bagian yang sengaja tidak lengkap.