Beberapa tim di perusahaan saya mempraktikkan alur kerja tinjauan kode yang belum pernah saya lihat sebelumnya. Saya mencoba memahami pemikiran di baliknya, dengan gagasan bahwa ada nilai dalam membuat seluruh perusahaan konsisten. (Saya berkontribusi pada banyak basis kode dan telah tersandung oleh perbedaan di masa lalu.)
- Penulis kode mengajukan permintaan tarik
- Reviewer memeriksa kodenya
- Jika peninjau menyetujui, mereka memberikan komentar di sepanjang baris "Terlihat bagus, jangan ragu untuk bergabung"
- Jika pengulas memiliki masalah, mereka memberikan komentar seperti "Harap perbaiki masalah kecil X dan Y, lalu gabungkan" (Untuk perubahan besar, kembali ke langkah 2)
- Pembuat kode melakukan perubahan jika perlu, dan kemudian menggabungkan permintaan tariknya sendiri
Saya memiliki masalah berikut:
Dalam hal persetujuan pada langkah 3, alur kerja ini membuat bolak-balik yang tampaknya tidak perlu ke penulis permintaan tarik. Peninjau, yang sudah melihat kode, bisa langsung menggabungkannya.
Dalam kasus perubahan yang diminta pada langkah 3, agensi untuk menggabungkan permintaan tarik sekarang sepenuhnya berada di tangan penulis PR. Tidak seorang pun selain penulis akan melihat perubahan sebelum penggabungan.
Apa beberapa keuntungan atau kerugian lain dari alur kerja ini? Apakah alur kerja ini umum di tim teknik lain?