Mendapatkan umpan balik dengan permintaan pull

Permintaan pull mendukung peninjauan dan penggabungan kode ke dalam satu proses kolaboratif. Setelah pengembang menambahkan fitur atau perbaikan bug, mereka membuat permintaan pull untuk memulai proses penggabungan perubahan ke cabang upstream. Anggota tim lain kemudian diberi kesempatan untuk meninjau dan menyetujui kode sebelum diselesaikan. Gunakan permintaan pull untuk mengulas pekerjaan yang sedang berlangsung dan mendapatkan umpan balik awal tentang perubahan. Tetapi tidak ada komitmen untuk menggabungkan perubahan. Pemilik dapat meninggalkan permintaan pull kapan saja.

Meminta agar kode ditinjau

Tinjauan kode yang dilakukan sebagai bagian dari permintaan pull bukan hanya untuk menemukan bug yang jelas; Itulah tujuan tesnya. Tinjauan kode yang baik menangkap masalah yang kurang jelas yang dapat menyebabkan masalah mahal nanti.

Ulasan kode membantu melindungi tim dari penggabungan yang buruk dan build yang rusak yang melemahkan produktivitas tim. Ulasan menangkap masalah sebelum penggabungan, melindungi cabang penting dari perubahan yang tidak diinginkan.

Tinjauan kode juga mendorong dan memperkuat kolaborasi dan komunikasi antara pengembang. Dan tim mendapatkan riwayat yang jelas dari semua perubahan yang dilakukan antara cabang utama dan cabang fitur.

Keahlian lintas penyerbukan dan menyebarkan strategi pemecahan masalah dengan menggunakan berbagai peninjau dalam ulasan kode. Keterampilan dan pengetahuan yang menyebar membuat tim lebih kuat dan lebih tangguh.

Berikan umpan balik yang bagus

Ulasan berkualitas tinggi dimulai dengan umpan balik berkualitas tinggi. Kunci umpan balik yang bagus dalam permintaan tarik adalah:

• Minta orang yang tepat untuk meninjau permintaan tarik.
• Pastikan pengulas tahu apa yang dilakukan kode tersebut.
• Berikan umpan balik yang konstruktif dan dapat ditindaklanjuti.
• Balas komentar tepat waktu.

Saat Anda menetapkan peninjau ke permintaan pull, pastikan untuk memilih set peninjau yang tepat. Peninjau harus tahu cara kerja kode, tetapi juga menyertakan pengembang yang bekerja di area lain sehingga mereka dapat berbagi ide.

Berikan deskripsi yang jelas tentang perubahan dan berikan build kode yang memiliki perbaikan atau fitur yang berfungsi di dalamnya. Peninjau harus berupaya memberikan umpan balik tentang perubahan yang tidak disetujui. Identifikasi masalah dan berikan saran khusus tentang apa yang dapat dilakukan secara berbeda. Umpan balik ini memiliki niat yang jelas dan mudah dipahami oleh pemilik pull request.

Pemilik permintaan pull harus membalas komentar, menerima saran, atau menjelaskan mengapa mereka menolak untuk menerapkannya. Beberapa saran baik, tetapi mungkin berada di luar cakupan permintaan pull. Ambil saran ini dan buat item kerja baru dan cabang fitur terpisah dari permintaan tarik untuk membuat perubahan tersebut.

Lindungi cabang dengan kebijakan

Ada beberapa cabang penting dalam repositori yang diandalkan tim selalu dalam kondisi yang baik, seperti main cabang. Teams dapat memerlukan permintaan pull untuk membuat perubahan apa pun pada cabang-cabang ini dengan platform seperti GitHub dan Azure DevOps. Pengembang yang mendorong perubahan secara langsung ke cabang yang dilindungi akan ditolak.

Tambahkan ketentuan tambahan ke permintaan pull untuk menerapkan tingkat kualitas kode yang lebih tinggi di cabang utama. Build kode gabungan yang jelas dan persetujuan dari beberapa peninjau adalah beberapa persyaratan tambahan yang dapat sering digunakan untuk melindungi cabang utama.

Pelajari selengkapnya

GitHub memiliki dokumentasi ekstensif tentang cara mengusulkan perubahan pada pekerjaan Anda dengan permintaan pull.

Baca selengkapnya tentang memberikan umpan balik yang bagus dalam peninjauan kode dan menggunakan template permintaan pull untuk memberikan panduan kepada peninjau Anda. Azure DevOps juga menawarkan pengalaman permintaan pull yang kaya yang mudah digunakan dan diskalakan sesuai kebutuhan.