Untuk "perubahan" yang terjadi pada commit, PR, atau MR kali ini, cukup mudah untuk meninggalkan komentar tentang apakah perubahan itu baik atau tidak. Namun, menyadari bagian yang seharusnya diubah tetapi tidak diubah lalu meninggalkan komentar tentangnya terasa tetap sulit bagi saya, meskipun saya sudah cukup lama melakukan code review. Terlebih lagi jika itu bukan codebase yang sangat saya kenal.
Saya bisa memikirkan beberapa cara, tetapi rasanya akan lebih baik jika ada lebih banyak mekanisme. Saya penasaran bagaimana kalian mengatasinya.
- Saat mengimplementasikan: rancang agar bagian yang nantinya harus diubah bersama bisa ikut tercakup oleh kode pengujian.
- Saat menulis PR: tambahkan pertanyaan yang relevan untuk dipertimbangkan reviewee di template PR. (misalnya, adakah PR sebelumnya dengan tujuan yang mirip dengan PR ini? Jika dibandingkan dengan PR tersebut, adakah bagian yang mungkin terlewat di PR ini?)
- Setiap kali terjadi hal "seharusnya dilakukan tetapi terlewat" di codebase tersebut, catatlah, lalu perbarui template PR dengan contoh itu setiap kali.
- Saat mereview PR: tambahkan ke panduan review agar reviewer mencari bagian yang terlewat dengan cara serupa dengan pertanyaan di atas.
Belum ada komentar.