Dokumen ini berisi petunjuk tentang cara meninjau perubahan. Perhatikan kriteria berikut untuk perubahan yang valid:
- Apakah perubahan ini sesuai dengan tujuan yang dinyatakan dalam project ini?
- Apakah perubahan ini valid dalam arsitektur project yang ada?
- Apakah perubahan ini menyebabkan kekurangan desain yang dapat menyebabkan masalah di masa mendatang?
- Apakah perubahan ini mengikuti praktik terbaik yang telah ditetapkan untuk project ini?
- Apakah perubahan ini merupakan cara yang baik untuk menjalankan fungsi yang dijelaskan?
- Apakah perubahan ini menimbulkan risiko keamanan atau ketidakstabilan, atau merusak aplikasi yang ada?
Untuk informasi tambahan tentang meninjau perubahan, lihat bagian Meninjau perubahan dalam dokumentasi Gerrit.
Jika Anda ditugaskan sebagai peninjau untuk perubahan, Anda akan menerima email dengan link ke perubahan tersebut. Ikuti langkah-langkah berikut untuk meninjau dan meninjau perubahan:
Klik link email ke perubahan. Layar Change Info Gerrit akan ditampilkan. Layar ini memberikan deskripsi perubahan, mengidentifikasi kontributor perubahan dan semua peninjau, serta memberikan daftar file yang terkait dengan perubahan tersebut.
Baca deskripsi perubahan untuk memahami tujuan perubahan.
Klik file untuk meninjau perubahan dalam file. File akan terbuka di layar perbandingan dua panel dengan file yang ada di sebelah kiri dan file dengan perubahan yang diusulkan di sebelah kanan. Untuk meninjau file:
Gunakan mouse untuk menandai (klik + tarik) bagian kode yang memiliki komentar atau pertanyaan. Pesan "Tekan c untuk memberi komentar" akan muncul.
Tekan c. Kotak komentar draf akan muncul.
Masukkan komentar atau pertanyaan Anda, lalu klik SIMPAN.
Klik nomor unik perubahan (sisi kiri atas layar) untuk kembali ke layar Info Perubahan.
Lanjutkan meninjau setiap file dalam perubahan.
Setelah selesai meninjau semua file dalam perubahan, kembali ke layar Info Perubahan dan klik BALAS. Layar Kirimkan suara persyaratan akan muncul.
Pilih salah satu skor berikut untuk perubahan tersebut:
- +1 Menurut saya bagus, tetapi orang lain juga harus menyetujuinya.
0 Tidak ada skor
-1 Sebaiknya tidak dikirimkan apa adanya, ada hal yang perlu diperbaiki.
-2 Perubahan ini tidak boleh dikirim
Klik BALAS untuk mengirim komentar dan skor Anda kepada developer perubahan.
Jika ada masalah yang perlu diperbaiki, developer akan memperbaikinya dan meminta Anda untuk meninjau perbaikan tersebut. Jika tidak, Anda telah selesai meninjau perubahan.