Dari apa yang saya lihat, sebagian besar langkah-langkah itu dilakukan pada Github dengan konvensi, dan bukan oleh proses resmi yang disediakan Github.
Majikan saya menggunakan Github, saya menjalankan sejumlah besar proyek open source kecil, dan membuat kontribusi sesekali ke proyek open source lainnya.
Inilah yang biasanya saya lihat dilakukan:
Penulis menambahkan kolega sebagai peninjau:
Ini bervariasi dari proyek ke proyek, tetapi secara umum, peer reviewer yang ditugaskan adalah semua kontributor untuk proyek .
Proyek open source tampaknya memiliki hierarki kasar - mungkin konvensi mereka hanya akan bergabung setelah kontributor "inti" telah memberikan apa-apa.
Di toko tempat saya bekerja saat ini, kami bergabung setelah salah satu dari setengah lusin pengembang di tim telah memberikan persetujuan mereka.
Terkadang seseorang dalam tim dapat menggunakan komentar untuk secara khusus memanggil pengembang lain yang mereka pikir harus meninjau kode sebelum bergabung, tetapi jika tidak, siapa pun yang sampai di sana terlebih dahulu dan merasa seperti melakukan itu dapat meninjau dan membuat komentar.
Persetujuan pengulas:
Persetujuan biasanya ditunjukkan dengan membuat komentar pada permintaan tarikan yang mengatakan "+1" atau "lgtm" (terlihat bagus untuk saya).
Tugas ringan:
Saya telah menggunakan kotak centang juga, tetapi dalam kebanyakan kasus, setiap komentar pada permintaan tarikan dianggap sebagai "tugas" implisit yang diselesaikan baik dengan:
- mengubah kode yang dikomentari oleh baris tersebut
- merespons dengan komentar lain
Melihat sekilas apa yang disetujui dan apa yang masih perlu ditinjau:
Saya telah menggunakan ekstensi Looks Good To Me untuk Chrome, yang memberi Anda pandangan seperti itu dari layar Tarik Permintaan. Tampilan daftar permintaan tarik tampaknya telah rusak oleh perubahan Github baru-baru ini.