Dokumen ini memberikan petunjuk tentang cara meninjau perubahan. Perhatikan kriteria berikut untuk perubahan yang valid:
- Apakah perubahan ini sesuai dengan tujuan yang dinyatakan proyek ini?
- Apakah perubahan ini valid dalam arsitektur proyek yang ada?
- Apakah perubahan ini menimbulkan kelemahan desain yang dapat menyebabkan masalah pada masa mendatang?
- Apakah perubahan ini mengikuti praktik terbaik yang telah ditetapkan untuk proyek ini?
- Apakah perubahan ini adalah 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 cara meninjau perubahan, lihat bagian Meninjau perubahan dalam dokumentasi Gerrit.
Jika Anda ditugaskan untuk menjadi peninjau untuk suatu perubahan, Anda akan menerima email yang berisi link ke perubahan tersebut. Ikuti langkah-langkah berikut untuk meninjau dan meninjau perubahan:
Klik link email untuk mengubah perubahan. Layar {i>Change Info<i} dari Gerrit akan ditampilkan. Layar ini menyediakan deskripsi perubahan, mengidentifikasi kontributor perubahan dan semua peninjau, serta menyediakan daftar file yang terkait dengan perubahan.
Baca deskripsi perubahan untuk memahami tujuan perubahan.
Klik file untuk meninjau perubahan pada file. File akan terbuka dalam 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 Anda. Pesan "Tekan c untuk berkomentar" akan muncul.
Tekan c. Kotak komentar draf akan muncul.
Masukkan komentar atau pertanyaan, 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 Ubah Info, lalu klik BALAS. Layar pemberian suara terkait persyaratan Kirim akan muncul.
Pilih salah satu skor berikut untuk perubahan:
- +1 Tampaknya bagus, tapi orang lain juga harus menyetujuinya.
0 Tidak ada skor
-1 Saya lebih suka kiriman ini tidak dikirimkan sebagaimana adanya, ada beberapa hal yang harus diperbaiki.
-2 Perubahan ini tidak boleh dikirimkan
Klik BALAS untuk mengirimkan komentar dan skor Anda kepada developer perubahan tersebut.
Jika ada masalah yang perlu diperbaiki, developer akan memperbaikinya dan meminta Anda untuk meninjau perbaikan tersebut. Jika tidak, Anda sudah selesai meninjau perubahan.