TBD.
TBD.
- Commit message:
- Use PascalCase for feature name
- Commit message using english
- Use the feature name prefix
- Example
FeatureName: message
- Saat membuat Pull Request (PR), buat deskripsi yang setidaknya mengandung hal-hal berikut:
- Tujuan PR, misalnya implementasi fitur baru atau bugfix. Jika perlu, tambahkan screenshot atau tabel yang dapat membantu menjelaskan PR tersebut.
- URL ke ticket yang membahas PR tersebut (jika tim menggunakan issue/task management tools).
- Mention setidaknya satu anggota tim dan satu engineer di luar tim (biasanya tech lead).
- Jika ada feedback untuk revisi kode, dan jika revisi sudah dilakukan, mention kembali pihak yang terlibat dalam diskusi feedback tersebut.
- PR hanya dapat di-merge jika sudah di-approve oleh setidaknya satu anggota tim dan satu engineer di luar tim (biasanya tech lead).
- Setelah source branch yang digunakan untuk PR berhasil di-merge ke target branch (misalnya branch
development
), hapus branch tersebut. Langkah ini tidak perlu dilakukan jika source branch yang digunakan adalahdevelopment
, dan target branch adalahmaster
.
- Ada setting yang dapat digunakan untuk menjadikan branch
development
danmaster
sebagai protected branch, mencegah branch tersebut agar tidak sengaja terhapus.- Protected branch di GitHub (Saat ini tidak tersedia untuk private repo yang menggunakan GitHub Free plan)
- Protected branch di GitLab
Code review adalah proses di mana kode yang dibuat oleh seorang engineer di-review oleh engineer lainnya. Code review pada umumnya dilakukan sebelum feature branch di-merge ke target branch (development
atau master
).
Tujuan dari code review adalah:
- Meningkatkan code quality dengan cara memperbaiki code style dan memperbaiki bagian kode yang berpotensi menimbulkan bug
- Mendapatkan implementasi logic terbaik untuk menyelesaikan suatu masalah
- Meningkatkan code ownership di antara anggota tim, terutama jika setiap anggota tim mengerjakan fitur yang berbeda dan menggunakan file yang berbeda.
TBD.
TBD.
TBD.