Adakah masalah atau pertimbangan yang melekat dengan mengangkat tiket di belakang ulasan, bukannya gagal?
Tidak secara inheren. Sebagai contoh, implementasi dari perubahan saat ini mungkin telah menggali masalah yang sudah ada di sana, tetapi tidak diketahui / nampak sampai sekarang. Gagal tiket tidak adil karena Anda akan gagal untuk sesuatu yang tidak terkait dengan tugas yang sebenarnya dijelaskan.
dalam ulasan kami menemukan suatu fungsi
Namun, saya menduga bahwa fungsi di sini adalah sesuatu yang ditambahkan oleh perubahan saat ini. Dalam hal ini, tiket harus gagal karena kode tidak lulus tes bau.
Di mana Anda akan menarik garis, jika tidak di mana Anda sudah menariknya? Anda jelas tidak berpikir kode ini cukup bersih untuk tetap berada dalam basis kode dalam bentuk saat ini; jadi mengapa Anda kemudian mempertimbangkan untuk memberikan tiket?
Gagal meninjau kode, sehingga tiket tidak ditutup dalam sprint ini, dan kami sedikit mendapat semangat, karena kami tidak dapat melewatkan tiket.
Tampaknya bagi saya seolah-olah Anda secara tidak langsung berpendapat bahwa Anda mencoba memberikan tiket ini suatu keuntungan bagi moral tim, daripada menguntungkan kualitas basis kode.
Jika itu masalahnya, maka prioritas Anda beragam. Standar kode bersih tidak boleh diubah hanya karena itu membuat tim lebih bahagia. Kebenaran dan kebersihan kode tidak bergantung pada suasana hati tim.
Refactor adalah pekerjaan kecil, dan akan selesai dalam sprint berikutnya (atau bahkan sebelum dimulai) sebagai cerita setengah poin yang sangat kecil.
Jika implementasi tiket asli menyebabkan bau kode, maka itu harus diatasi di tiket asli. Anda hanya boleh membuat tiket baru jika bau kode tidak dapat secara langsung dikaitkan dengan tiket asli (misalnya, skenario "jerami yang mematahkan punggung unta").
Sumber daya yang saya dapat temukan dan telah membaca ulasan kode detail 100% atau tidak sama sekali, biasanya, tetapi saya menemukan itu biasanya tidak realistis.
Lulus / gagal pada dasarnya adalah keadaan biner , yang secara inheren semua atau tidak sama sekali.
Apa yang Anda maksudkan di sini, saya pikir, adalah lebih dari yang Anda menafsirkan ulasan kode memerlukan kode sempurna atau jika gagal, dan itu tidak terjadi.
Kode tidak boleh rapi, hanya harus memenuhi standar kebersihan yang masuk akal yang digunakan oleh tim / perusahaan Anda. Ketaatan pada standar itu adalah pilihan biner: ia menganut (lulus) atau tidak (gagal).
Berdasarkan uraian masalah Anda, jelas bahwa Anda tidak berpikir bahwa ini mematuhi standar kode yang diharapkan, dan karenanya tidak boleh dilewatkan karena alasan tersembunyi seperti moral tim.
Kalau tidak, tugas tersebut sesuai dengan definisi yang dilakukan.
Jika "itu menyelesaikan pekerjaan" adalah tolok ukur terbaik untuk kualitas kode, maka kita tidak perlu menciptakan prinsip kode bersih dan praktik yang baik untuk memulai - kompiler dan pengujian unit akan menjadi proses peninjauan otomatis kami dan Anda tidak perlu ulasan kode atau argumen gaya.