Review zorunlu
- Git kursu
- İleri seviye
- Ders 50/56
- 4 dk
Zorunlu inceleme kuralını öğrenirsin.
Zorunlu inceleme nedir?
Bir pull request'in birleştirilmeden önce başka bir ekip üyesi tarafından incelenmesini zorunlu hale getirebilirsin. GitHub'da bu kural, belirlenen sayıda onay alınmadan değişikliklerin korunmuş dala birleştirilmesini engeller. Böylece kod değişiklikleri ikinci bir göz tarafından kontrol edilir.
main:
required_approvals: 1Özellikle ekip çalışmalarında hataları erken fark etmeye yardımcı olur.
Onay ve inceleme politikası
Zorunlu inceleme kuralında kaç onay gerektiğini belirleyebilirsin. Örneğin required_approvals: 1 en az bir onay ister. GitHub ayrıca pull request'i açan kişinin kendi değişikliğini onaylamasını geçerli bir ekip onayı olarak kabul etmez.
pull_request:
required_approvals: 1
dismiss_stale_reviews: truedismiss_stale_reviews seçeneği, incelemeden sonra yeni commit geldiğinde eski onayların geçersiz sayılmasını sağlar.
Uygulama İki onay zorunlu olsun
main dalı için pull request inceleme politikasını yapılandır. required_approvals değerini 2 yap ve yeni commit geldiğinde eski onayları geçersiz kılmak için dismiss_stale_reviews değerini true olarak ayarla.
Başlangıç kodu
# İnceleme politikası oluştur
# Gerekli onay sayısını belirle
# Eski onayları geçersiz kılÖrnek çözüm
main:
pull_request:
required_approvals: 2
dismiss_stale_reviews: trueBeklenen çıktı
İki onay zorunlu hale getirildi.
Soru ve cevaplar
Dersin sorularını cevaplarıyla birlikte inceleyebilirsin. Kendini denemek istersen dersi interaktif olarak çöz.
1 Bir pull request'in birleştirilmesi için en az iki ekip üyesinin onayı isteniyorsa hangi değer kullanılmalıdır?
- required_approvals: 2 Doğru cevap
- required_approvals: 0
- required_approvals: 1
- required_approvals: false
Açıklama: İki ayrı onay gerektiğinde <code>required_approvals</code> değeri 2 olarak belirlenir.
2 Eski onayları geçersiz kıl
pull_request:
required_approvals: 1
____: truepull_request:
required_approvals: 1
dismiss_stale_reviews: trueAçıklama: Yeni commit geldiğinde önceki incelemelerin geçersiz sayılması için <code>dismiss_stale_reviews</code> kullanılır.
3 İnceleme kuralları
| Kavram | Anlamı |
|---|---|
Approval |
Değişikliğe verilen onay |
required_approvals |
Gerekli onay sayısı |
dismiss_stale_reviews |
Eski onayları geçersiz kılar |
Pull request yazarı |
Kendi onayını saydıramaz |