Kod İnceleme
Denomas kod inceleme kültürü ve süreci — MR standartları, review kontrol listesi, onay kuralları
2 minute read
Kod İnceleme
Kod inceleme, Denomas mühendislik kültürünün temel taşlarından biridir. Her değişiklik, merge edilmeden önce en az bir takım arkadaşının incelemesinden geçer.
İlkeler
- Her MR Review Alır — İstisna yoktur, tek satırlık değişiklikler bile
- Yapıcı Geri Bildirim — Eleştiri koda yöneliktir, kişiye değil
- Öğrenme Aracı — Review, hem yazar hem reviewer için öğrenme fırsatıdır
- Otomasyon Desteği — Linter ve testler review öncesi otomatik çalışır
Merge Request Standartları
Her MR şu bilgileri içermelidir:
## Ne Değişti?
[Değişikliğin kısa açıklaması]
## Neden?
[Motivasyon — hangi sorunu çözüyor veya hangi özelliği ekliyor]
## Nasıl Test Edildi?
[Test adımları veya otomatik test referansları]
## Kontrol Listesi
- [ ] Pre-commit hook'ları geçti
- [ ] Birim testleri eklendi/güncellendi
- [ ] Dokümantasyon güncellendi
- [ ] CHANGELOG güncellendi (gerekiyorsa)
Review Kontrol Listesi
Reviewer şu noktaları değerlendirir:
| Kategori | Kontrol |
|---|---|
| Doğruluk | Kod beklenen işi yapıyor mu? |
| Tasarım | Mimari prensiplere uygun mu? |
| Okunabilirlik | Kod kendini açıklıyor mu? |
| Güvenlik | Güvenlik açığı var mı? |
| Performans | Performans etkisi kabul edilebilir mi? |
| Test | Yeterli test kapsaması var mı? |
| Dokümantasyon | Handbook güncellenmesi gerekiyor mu? |
Onay Kuralları
| Değişiklik Türü | Minimum Onay | Gerekli Reviewer |
|---|---|---|
| Bug fix | 1 onay | Herhangi bir mühendis |
| Yeni özellik | 2 onay | En az 1 senior mühendis |
| Mimari değişiklik | 2 onay + architect | Lead + Architect |
| Güvenlik ile ilgili | 2 onay + security | @guvenlik-ekibi temsilcisi |
Hızlı Review İçin
MR'ınızı küçük tutun. 400 satırdan fazla değişiklik içeren MR'lar bölünerek gönderilmelidir. Küçük MR'lar daha hızlı ve etkili review alır.
AI Review
Yapay Zeka Asistanları, yetkinlik seviyelerine uygun olarak code review görevlerinde bulunabilir. Denomas Yetkinlik Çerçevesi seviye 3+ Yapay Zeka Asistanları otomatik ön-review yapabilir.
İlgili Sayfalar
- Mühendislik Genel Bakış
- Kod Kalitesi
- CI/CD Pipeline
- Mühendislik
Geribildirim
Bu sayfa yararlı oldu mu?
Bu sayfa faydali oldu
Bu sayfa gelistirilmeli
