Dokumen ini berisi 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 project yang ada?
- Apakah perubahan ini menyebabkan kekurangan desain yang dapat menyebabkan masalah di masa mendatang?
- Apakah perubahan ini mengikuti praktik terbaik yang telah dibuat untuk proyek 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 cara meninjau perubahan, lihat Meninjau perubahan dokumentasi Gerrit.
Jika 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 dua panel perbandingan dengan file yang ada di sebelah kiri dan file dengan perubahan di sebelah kanan. Untuk meninjau file:
Gunakan mouse untuk menandai (klik + tarik) bagian kode yang memiliki komentar atau pertanyaan. Pesan "Tekan c untuk berkomentar" 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 Ubah Info.
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 Tampaknya bagus, tapi 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 dikirimkan
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.