Bagaimana cara melakukan peer review pada permintaan tarik GitHub?

12

Kami pindah dari Bitbucket ke GitHub dan satu hal yang kami perjuangkan adalah ulasan kode rekan yang bekerja sangat lancar di Bitbucket seperti ini:

  1. Penulis membuka Permintaan Tarik (GitHub: sama)
  2. Penulis menambahkan kolega-koleganya sebagai pengulas (GitHub: ?? berjuang di sini dengan banyak orang yang ditugaskan)
  3. Peninjau juga:
    1. Menyetujui PR dengan tanda centang hijau (GitHub: ??)
    2. Komentar yang ditambahkan (GitHub: sama)
    3. Tugas ringan yang dibuat (GitHub: semacam mirip jika - [ ]sintaks digunakan dalam deskripsi PR; memalukan bahwa itu tidak bekerja untuk tugas)
  4. Ada daftar PR di mana saya bisa melihat sekilas yang ditinjau dan OK untuk digabung dan yang perlu perhatian lebih lanjut (GitHub: ??)

Saya harus menunjukkan bahwa kami ingin menghindari alat peninjau kode pihak ke-3 jika memungkinkan dan ingin tetap menggunakan vanilla GitHub dengan beberapa solusi.

code-reviews github pull-requests Borek Bernard
sumber
1
Sepertinya Anda mungkin beralih sebelum waktunya. Mengapa tetap beralih, terutama jika hal baru itu tidak memiliki semua fitur yang Anda butuhkan?
pengasuh
Tulis komentar ke prq Anda, dan sorot @ siapa pun yang ingin mendapat pemberitahuan. Peninjau dapat menambahkan tag untuk menunjukkan pendapat ulasan mereka.
Wilbert 2-15
Perhatikan juga bahwa Anda dapat @ menyebutkan seluruh tim, misalnya @ MyCompany / developer
Whymarrh

Jawaban:

6

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.

TehShrike
sumber