Di GitHub, apa perbedaan antara reviewer dan penerima pengalihan?

Sebuah fitur yang ditambahkan pada 7 Desember 2016, diumumkan di blog GitHub, memperkenalkan opsi untuk menambahkan pengulas ke Permintaan Tarik

Anda sekarang dapat meminta ulasan secara eksplisit dari kolaborator, membuatnya lebih mudah untuk menentukan siapa yang ingin Anda tinjau permintaan tarik Anda.

Anda juga dapat melihat daftar orang-orang yang sedang Anda tunggu ulasannya di sidebar halaman permintaan tarik, serta status ulasan dari mereka yang sudah meninggalkannya.

Namun, pengaturan eksplisit resensi untuk PR sudah dilakukan dengan menugaskan orang ( opsi penerima hak ).

Dengan kedua opsi yang sekarang tersedia, apa peran masing-masing opsi karena keduanya memiliki tujuan akhir yang sama?

EDIT:

Setelah berdiskusi dengan beberapa pengelola OSS, pengulas didefinisikan sebagai kata yang seharusnya: meninjau (kode seseorang) dan "penerima hak" memiliki definisi yang lebih longgar seperti dijelaskan di bawah ini.

Untuk "resensi" : seseorang yang ingin Anda tinjau kodenya. Belum tentu orang yang bertanggung jawab untuk area itu atau bertanggung jawab untuk menggabungkan komit. Bisa jadi seseorang yang mengerjakan potongan kode itu sebelumnya, seperti yang disarankan GitHub.

Untuk "penerima hak" : hingga tim proyek / pengelola apa artinya dan tidak ada definisi yang ketat. Ini bisa menjadi pembuka PR, atau seseorang yang bertanggung jawab untuk area itu (siapa yang akan menerima PR setelah ulasan selesai atau tutup saja). GitHub tidak dapat menentukan apa yang dibiarkan terbuka untuk pengelola proyek yang paling cocok untuk proyek mereka.

Jawaban sebelumnya:

Ok saya akan pergi ke depan dan menjawab pertanyaan saya sendiri.

Untuk PR pengguna dengan akses tulis: Penerima akan menjadi orang yang sama yang membuka PR, dan peninjau akan mengganti fungsi penerima yang lama (meninjau kode), karena ini adalah seseorang yang dipilih oleh penerima yang ditunjuk.

Untuk PR pengguna tanpa akses tulis (kontributor dari luar): Seseorang dengan akses tulis akan menugaskan dirinya sendiri (atau anggota hak istimewa menulis lainnya), untuk meninjau kembali PR (Reviewer). Penerima kosong.

Untuk PR yang belum selesai dari kontributor luar : anggota akses tulis akan mengambil pekerjaan yang belum selesai dan menugaskannya. Dia akan bertanggung jawab untuk menyelesaikan tugas, menjadi Assignee . Karena alasan utama PRs meninjau perubahan, dia akan memilih beberapa orang lain untuk meninjau perubahan.

Di GitHub, resensi adalah orang yang meninjau permintaan tarik. Pemilik proyek dapat meminta peninjauan dari pengelola mana pun, Mereka bahkan dapat menetapkan opsi sehingga permintaan tarik hanya dapat digabung jika ditinjau oleh salah satu pengelola dengan akses tulis.

Menurut dokumentasi resmi github , Assignee adalah orang yang mengerjakan masalah spesifik dan menarik permintaan. Kadang-kadang bingung sebagai peninjau. Ini sebenarnya dimaksudkan untuk digunakan dengan masalah daripada menarik permintaan sehingga ketika kami menerima masalah kami dapat menugaskan seseorang untuk memperbaikinya. Dalam permintaan tarikan, penerima hak merujuk pada seseorang yang bertugas menggabungkan permintaan tarikan itu setelah mendapat komentar dan mengubah permintaan dari pengelola lain.

Sesuai jawaban yang diterima. Ya, "penerima hak" memiliki definisi yang lebih longgar dan dapat digunakan secara berbeda sesuai dengan kebutuhan tim.

Dalam tim kami yang terdiri dari 8 devs, di sebagian besar PR kami memiliki 1 reviewer, yang menyarankan perubahan dan akhirnya menyetujui PR. Selama fase peninjauan, "penerima hak" adalah orang yang membuka PR; nanti jika PR diambil oleh pengembang lain, "penerima hak" baru ditambahkan. Setelah PR disetujui dan siap untuk QA atau penggabungan langsung, "penerima" QA baru ditambahkan. Dengan cara ini daftar "penerima hak" tumbuh.

Kami menggunakan "penerima hak" untuk menunjuk orang-orang berikut secara kolektif:

1. Tarik Permintaan Penulis
2. Penulis mengerjakan saran perubahan PR (biasanya sama dengan 1)
3. QA orang yang terlibat
4. Orang yang bertanggung jawab untuk bergabung (biasanya sama dengan 2 atau 3)

Menggunakan "penerima hak" membantu menemukan PR di masa depan dengan mudah. Salah satu proyek saya memiliki> 3000 PR.

is:open is:pr author:raya-dumas

is:closed is:pr assignee:raya-dumas

Atau hanya author:raya-dumas untuk menemukan semua item yang dibuat penulis (masalah, PR)

dan pertanyaan serupa lainnya untuk memudahkan proses pencarian. "tonggak" cukup membantu untuk digunakan juga untuk memudahkan pencarian PR.

Sebelum GitHub hanya memiliki bidang yang ditugaskan dan tidak ada bidang resensi. Tidak ada perbedaan saat itu sehingga bidang yang ditugaskan paling umum digunakan sebagai bidang resensi.

Tetapi gunakan mereka dengan cara apa pun yang sesuai dengan proyek Anda.