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 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 ditetapkan sebagai peninjau perubahan, Anda akan menerima email berisi link ke perubahan tersebut. Ikuti langkah-langkah berikut untuk meninjau dan meninjau perubahan:
Klik link email ke perubahan. Layar Info Perubahan 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 ingin Anda komentari atau tanyakan. 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, lalu klik BALAS. Layar Kirim suara persyaratan akan muncul.
Pilih salah satu skor berikut untuk perubahan:
- +1 Terlihat bagus, tetapi orang lain juga harus menyetujui.
0 Tidak ada skor
-1 Saya lebih suka jika ini tidak dikirimkan apa adanya, ada hal yang perlu diperbaiki.
-2 Perubahan ini tidak boleh dikirimkan
Klik BALAS untuk mengirimkan 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.