
PR pertama terasa besar walaupun hanya menukar label borang. Kita bimbang reviewer menemui kesilapan mudah. Hakikatnya, perubahan kecil ialah tempat terbaik untuk belajar aliran pasukan: branch, commit, ujian, review dan deploy. Tujuannya bukan menunjukkan kod sempurna pada percubaan pertama.
Pastikan PR mempunyai tajuk yang menerangkan masalah, ringkasan apa berubah dan cara anda mengujinya. Jika ada tangkap layar yang membantu, sertakan. Nyatakan perkara yang anda kurang pasti supaya reviewer tahu di mana perlu melihat lebih dekat. Jangan masukkan fail yang tidak berkait hanya kerana terubah ketika setup lokal.
Jawab review dengan rasa ingin tahu
Jika ada cadangan, baca sebabnya. Tanya apabila komen kabur dan cuba terangkan apa yang anda faham sebelum mengubah kod. Ada komen yang wajib kerana risiko, ada yang hanya gaya pasukan. Setelah dikemas kini, uji semula dan beritahu reviewer perubahan yang dibuat. Panduan GitHub tentang pull request menerangkan struktur asas perbualan itu.
Sebelum meminta review, buka diff PR seperti orang lain akan melihatnya. Adakah fail yang berubah sepadan dengan tujuan? Adakah terdapat log debug atau teks sementara? Semakan sendiri ini tidak menjamin tiada ralat, tetapi ia menghormati masa reviewer dan membuat perbualan lebih fokus pada keputusan penting.
Selepas merge, lihat hasil di persekitaran sebenar jika dibenarkan. Satu pembetulan kecil yang sampai kepada pengguna ialah pelajaran lengkap tentang kerja pembangunan, lebih berharga daripada menyimpan kod yang tidak pernah disemak.