Di GitHub, apa perbedaan antara reviewer dan penerima pengalihan?


187

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

Opsi Peninjauan GitHub

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?


1
kapan "fitur penerima hak" dirilis pertama kali? Apakah ada artikel yang memperkenalkannya?
babeyh

Jawaban:


135

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.


24
Untuk setiap anggota tim baru, saya harus mengirim tautan ke jawaban ini untuk menjelaskan cara berurusan dengan penerima tugas dan pengulas. Yang membuat saya berpikir bahwa ada sesuatu yang secara fundamental salah di sini :(
Andrey Kuleshov

Apakah penerima hak harus memiliki akses tulis?
Emre Sülün

apakah ada perbedaan perilaku pemberitahuan email antara keduanya?
jxramos

26

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.


2
Terima kasih atas jawabannya, tapi saya rasa ini tidak menjawab pertanyaan sepenuhnya. Anda dapat memberikan masalah kepada seseorang (jadi dia akan menjadi penerima masalah), tetapi ketika PR dikirim seseorang akan menjadi peninjau (penerima PR), dan pada titik ini, saya masih tidak jelas tentang perbedaan antara penerima hak dan pengulas
Cezar Augusto

14

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.

Tangkapan layar Github, Q4 2017


Sangat dijelaskan.
Nitin Gaur

Harus disebutkan bahwa Anda bisa mencari penulis: my-github-handle untuk menemukan apa yang diciptakan seseorang
Wisienkas

1

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.

Dengan menggunakan situs kami, Anda mengakui telah membaca dan memahami Kebijakan Cookie dan Kebijakan Privasi kami.
Licensed under cc by-sa 3.0 with attribution required.