Dokumen ini memberikan 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 menimbulkan kekurangan desain yang dapat menyebabkan masalah di masa mendatang?
- Apakah perubahan ini mengikuti praktik terbaik yang telah ditetapkan untuk proyek ini?
- Apakah perubahan ini merupakan cara yang baik untuk melakukan fungsi yang dijelaskan?
- Apakah perubahan ini menimbulkan risiko keamanan atau ketidakstabilan, atau merusak aplikasi yang ada?
Untuk mengetahui informasi tambahan tentang meninjau perubahan, lihat bagian Meninjau perubahan dalam dokumentasi Gerrit.
Jika Anda ditetapkan sebagai peninjau perubahan, Anda akan menerima email yang berisi link ke perubahan tersebut. Ikuti langkah-langkah berikut untuk meninjau dan meninjau perubahan:
Klik link email untuk melakukan perubahan. Layar Info Perubahan Gerrit ditampilkan. Layar ini memberikan deskripsi perubahan, mengidentifikasi kontributor perubahan dan semua peninjau, serta memberikan daftar file yang terkait dengan perubahan.
Baca deskripsi perubahan untuk memahami tujuan perubahan.
Klik file untuk meninjau perubahan dalam file. File 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 Anda untuk menandai (klik + tarik) bagian kode yang ingin Anda beri komentar atau ajukan pertanyaan. Pesan "Tekan c untuk berkomentar" 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 Kirim suara persyaratan akan muncul.
Pilih salah satu skor berikut untuk perubahan:
- +1 Terlihat bagus bagi saya, tetapi orang lain juga harus menyetujui.
0 Tidak ada skor
-1 Saya lebih memilih agar ini tidak dikirimkan apa adanya, ada hal-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.