Penafian: Ada beberapa pertanyaan serupa, tetapi saya tidak menemukan sentuhan mana yang secara khusus masalah yang Anda hadapi saat meninjau permintaan tarik besar.
Masalah
Saya merasa ulasan kode saya dapat dilakukan dengan cara yang lebih baik. Saya terutama berbicara tentang ulasan kode besar dengan banyak perubahan di 20+ file.
Cukup mudah untuk menangkap masalah kode lokal yang jelas. Memahami apakah kode tersebut memenuhi kriteria bisnis adalah cerita yang berbeda.
Saya mengalami kesulitan mengikuti proses pemikiran pembuat kode. Ini cukup sulit ketika perubahannya banyak dan menyebar di beberapa file. Saya mencoba untuk fokus pada grup file yang terkait dengan bagian perubahan tertentu. Kemudian tinjau kelompok satu per satu. Sayangnya alat yang saya gunakan (Atlassian Bitbucket) tidak terlalu membantu. Setiap kali saya mengunjungi file, itu ditandai seperti yang terlihat, meskipun sering ternyata tidak terkait dengan bagian perubahan yang saat ini diperiksa. Belum lagi beberapa file harus dikunjungi beberapa kali dan perubahannya diulas satu per satu. Juga kembali ke file yang relevan ketika Anda mengikuti jalan yang buruk tidak mudah.
Kemungkinan solusi, dan mengapa mereka tidak berhasil untuk saya
Meninjau permintaan tarik dengan komit sering kali memecahkan masalah ukuran, tapi saya tidak suka karena saya akan sering melihat perubahan yang sudah usang.
Tentu saja, membuat permintaan tarikan yang lebih kecil sepertinya obat, tetapi itulah masalahnya, terkadang Anda mendapatkan permintaan tarikan yang besar dan harus ditinjau.
Anda juga dapat mengabaikan aspek logis dari kode secara keseluruhan, tetapi tampaknya cukup berisiko, terutama ketika kode tersebut berasal dari seorang programmer yang tidak berpengalaman.
Menggunakan alat yang lebih baik bisa membantu, tetapi saya tidak menemukannya.
Pertanyaan
- Apakah Anda memiliki masalah yang serupa dengan ulasan kode Anda? Bagaimana Anda menghadapi mereka?
- Mungkin Anda memiliki alat yang lebih baik?