Materi
Sebuah review berakhir di salah satu dari tiga keadaan: Comment memberi masukan tanpa keputusan, Approve menyetujui, Request changes menahan penggabungan sampai masukannya ditangani. Masukan sering menempel pada satu baris tertentu, dan peninjau boleh melampirkan usulan perubahan berbentuk potongan kode jadi (suggested change) yang bisa diterima langsung dari halaman GitHub menjadi sebuah commit di cabang pull request itu.
Perbaikan berarti commit baru, bukan pull request baru
Pull request menunjuk sebuah cabang, bukan sekumpulan commit yang dibekukan. Selama commit perbaikan mendarat di cabang yang sama, commit itu ikut masuk ke pull request yang sudah terbuka — dokumentasi GitHub menyebutnya menambah commit baru ke head branch. Karena pasangan cabang sudah dicatat pada push pertama dengan -u, push berikutnya cukup ditulis git push.
echo "Setiap koreksi dicatat dengan tanggalnya." >> README.md
git add README.md
git commit -m "Tambah catatan koreksi sesuai review"
git pushPesan commit perbaikan menjelaskan perubahannya, bukan menyebut nama peninjau atau nomor komentarnya. Riwayat dibaca orang yang tidak pernah melihat percakapan review itu, sering bertahun-tahun kemudian, jadi Tambah catatan koreksi lebih jelas daripada Perbaiki sesuai masukan Budi.