Etiket untuk mengembalikan karya orang lain

Baru-baru ini saya berdebat dengan teman satu tim bahwa saya "tidak berkonsultasi dengan mereka sebelum kembali" karena itu membuat mereka "terlihat seperti orang idiot". (Untuk konteks, ini adalah proyek universitas, dan pengembaliannya lebih dari penempatan kode.)

Ini membuat saya bertanya-tanya: Apa norma untuk memberi tahu pengendara yang perlu dilakukan pemulihan? Bagaimana Anda memberi tahu mereka bahwa Anda perlu mengembalikan komit ini tanpa memicu perasaan keras?

Memiliki ulasan kode.

Itu seharusnya bisa menyelesaikan sebagian besar dari mereka, jika Anda menggunakan komit Anda masih dapat membuat permintaan tarik bahkan jika itu hanya untuk satu orang lain untuk meninjau tim Anda.

Jika kesalahan terjadi, maka itu adalah kesalahan tim dan bukan hanya satu orang.

Jika Anda tidak ingin melakukan itu karena suatu alasan, tidak ada cara mudah untuk melakukannya. Anda tidak memiliki proses untuk memasukkan kode ke dalam repo, maka, yah, jangan kaget ketika perasaan orang lain terluka.

Seperti yang sering terjadi, ketika Anda menghadapi situasi di mana Anda melakukan hal-hal yang akan membuat orang lain frustrasi, biasanya baik untuk berbicara dengan mereka terlebih dahulu sebelum melakukan tindakan. Dalam hal ini mungkin itu berarti berbicara dengan kolega Anda dan bertanya, "Saya berpikir untuk mengembalikan X karena [alasan] - apa pendapat Anda?"

Anda punya alasan bagus untuk mengembalikan komitnya, atau tidak. Jika Anda memiliki alasan yang bagus, maka "membuat saya terlihat seperti orang idiot" bukanlah argumen balasan yang baik. Yang mengatakan, akan sopan untuk memberi tahu orang tersebut terlebih dahulu tentang alasan untuk menghindari pertengkaran.

Dan semua yang dikatakan, akan jauh lebih baik untuk memiliki ulasan kode, jadi semoga kembali tidak boleh terjadi - mungkin terjadi komit ditolak, tetapi Anda tidak boleh masuk ke situasi di mana ia dikembalikan.

Pertanyaan mendasar untuk masalah ini adalah "siapa yang memiliki komponen pada tingkat teknis"?

Jika tidak ada jawaban untuk ini, atau aswernya adalah "kita semua melakukan" atau "tidak ada yang" atau Anda hanya mendapatkan tampilan bingung dan tidak ada yang melihat masalah, jangan buang energi Anda dan cari pekerjaan lain.

Jika ada jawaban dan pembicaraan dengan changer tidak mengarah pada kesepakatan, lakukan dengan pemilik dan changer, buat kasus Anda dan biarkan pemilik memutuskan.

Tanggung jawab bersama adalah akar dari semua basis kode yang menyedihkan dan karyawan yang menyedihkan.

Saya pikir itu adil untuk dimintai konsultasi jika seseorang memiliki investasi waktu yang signifikan dalam sesuatu, terutama jika itu masih baru. Lebih baik sampai pada konsensus terlebih dahulu. Idealnya Anda akan meyakinkan orang tersebut untuk melakukan pengembalian diri.

Jika Anda mengalami perselisihan yang tidak dapat Anda lakukan, berkonsultasilah dengan orang ketiga. Biarkan suara mayoritas menang, atau pastikan orang ketiga adalah pengembang utama.

Kadang-kadang Anda harus membiarkan barang pergi, Anda tidak harus memenangkan setiap pertempuran. Pastikan Anda didengar.

Saya pikir itu sehat untuk tidak menjadi melekat pada kode yang Anda tulis. Saya sering mengembalikan kode saya sendiri, orang lain harus dapat melakukan hal yang sama ketika diperlukan.

Jangan hanya kembali, biarkan mereka melakukan pengembalian sendiri.

Alasan umum yang dapat Anda berikan saat meminta pengembalian:

• "Komit XYZ Anda memecah ini dan itu penggunaan / fungsi / testcase"
• "Komit XYZ Anda tidak mematuhi standar pengkodean kami misalnya lekukan yang salah"
• "Komit Anda XYZ melanggar proses organisasi kami, mis. Komit ke komponen A harus ditinjau sebelum-komitmen oleh pemilik komponen B"

Hanya alasan untuk mengembalikan sesuatu yang tidak dilakukan sendiri

• Orang diperpanjang (liburan / sakit)
• Komit merusak bangunan atau melakukan sesuatu yang mencegah anggota tim lain dari melakukan pekerjaan mereka.
• Komitmen kereta sudah dalam produksi dan perlu diperbaiki secepatnya