Sepertinya tim tidak memiliki proses formal untuk tinjauan kode.
Saya tidak berbicara tentang membuat dokumen Word setebal 350 halaman, tetapi hanya beberapa poin sederhana tentang apa yang diperlukan prosesnya.
Bit penting:
Tentukan satu set inti dari pengulas. Tidak ada pernyataan umum. Sebutkan nama orang.
Ini harus menjadi pengembang senior Anda.
Diperlukan lebih dari 1 reviewer inti untuk menandatangani ulasan.
Identifikasi setidaknya 1 reviewer non inti lainnya setiap sprint atau rilis yang merupakan reviewer inti sementara. Mengharuskan mereka keluar dari semua ulasan kode selama waktu ini.
Butir # 3 memungkinkan pengembang lain untuk memutar ke kelompok resensi inti. Beberapa minggu mereka akan menghabiskan lebih banyak waktu untuk ulasan daripada yang lain. Itu adalah tindakan penyeimbangan.
Adapun orang yang memberi penghargaan? Berkali-kali mengakui upaya yang dilakukan seseorang selama peninjauan kode di depan seluruh tim dapat berhasil, tetapi jangan terlalu memaksakan diri untuk hal ini.
Jika ragu, tentukan prosesnya dan beri tahu tim bahwa mereka harus menaatinya.
Dan ada satu hal terakhir yang dapat Anda coba - mungkin kontroversial: biarkan @ # $% mengenai penggemar, jika saya dapat menggunakan idiom.
Biarkan tim gagal, karena proses peninjauan kode tidak diikuti. Manajemen akan terlibat, dan kemudian orang akan berubah. Ini benar-benar hanya ide yang bagus dalam kasus-kasus paling ekstrem di mana Anda telah mendefinisikan suatu proses dan tim menolak untuk mematuhinya. Jika Anda tidak memiliki wewenang untuk memecat orang atau mendisiplinkan mereka (karena kebanyakan pengembang utama tidak ) maka Anda perlu melibatkan seseorang yang dapat melakukan hal ini.
Dan tidak ada yang gagal untuk mengubah keadaan. Terlepas dari apa yang orang katakan, Anda dapat mengarahkan Titanic - tetapi tidak sebelum menghantam es es.
Terkadang Anda hanya perlu membiarkan Titanic menghantam burg ice.