İnceleme ve Değişiklik İzlenebilirliği

Neden önemli

Olay incelemesi değişikliği kimin neden onayladığını sorar. Bağlı çekme istekleriyle yanıt otuz saniyedir: fark, tartışma, kontroller, onaylayan. Onlar olmadan yanıt sohbet logları ve hafızada arkeolojidir; hafıza kaybeder. İzlenebilirlik denetçilere evrak değildir; ekibin kendi geçmişinden hızla öğrenme yetisidir.

Kavramlar

İncelenebilir çekme isteğinin dört parçası vardır: değişikliği adlandıran başlık, nedeni anlatan açıklama (sorun, yaklaşım, elenen alternatifler), küçük fark (tek fikir, test dahil) ve yeşil kontroller. Boyut baskın faktördür: birkaç yüz satırı aşınca inceleme kalitesi çöker; büyük işi her biri bağımsız tutarlı yığın veya ardışık isteklere böl. Taslak istekler süren işi işaretler; inceleme istekleri hazırlığı. İki sinyali asla karıştırma.

Makinelerin kontrol edemediğine incele: yaklaşımın doğruluğu, eksik uç durumlar, hata ele alma, güvenlik etkileri ve testlerin bir şey kanıtlayıp kanıtlamadığı. Tarz, biçim ve önemsiz notlar linter ve biçimlendiricilere aittir; otomatik dayatılır, insanlar inceleme bütçesini onlara harcamaz. Onaylamak, bunu anladım ve çalışmasının sorumluluğunu alıyorum demektir; yorum tartışalım, değişiklik isteği çözülmeden birleştirme demektir. Lastik mühürler tanınır: büyük farklarda yorumsuz onaylar, tekrar tekrar, aynı ikiliden.

İzlenebilirlik dört halkayı zincirler: üretim kodu commite (blame), commit çekme isteğine (istek numarasını anan mesaj), istek bilete (gerekçeli bağlı kayıt), bilet karara (neden bu yaklaşım). Zinciri kahramanlıkla değil gelenekle bütün tut: isteği özetleyen squash mesajları, kayıt numarası taşıyan dal adları, commiti adlandıran dağıtım kayıtları. Altı ay sonra herhangi satırda git blame kime, ne zaman, hangi istek ve neden sorularını insana sormadan yanıtlamalıdır.

Uygulamalı örnek

Tek turda onaylanan isteğin anatomisi:

Title: Cap login retries at 3 with backoff (fixes AUTH-412) Why: brute-force traffic tripled failed logins; unbounded retries amplify it. Chose 3 attempts (matches M03 retry budget lesson). Diff: 40 lines + 25 test lines, rate-limit path only. Checks: unit + integration green, ShellCheck clean.

Beklenen okuma: başlık kaydı bağlar, neden elenen alternatifi üstü kapalı kaydeder (sınırsız sınırsız kalır), fark kanıtlı tek fikirdir, kontroller insan bakmadan önce çalışmıştır. İnceleyen yaklaşımı dakikada doğrular, çünkü kontrol edilebilir her şey önceden kontrol edilmiştir. Karşılaştır: 2.000 satırlık various fixes başlıklı yeniden düzenleme; yapısı gereği incelenemez, bitkinlikten onaylanır, üretimde pişman olunur.

Yaygın yanlış hamle

Farkları açıklamayı okumadan yukarıdan satır satır incelemek. İncelemeyi redaksiyona çevirir: yazım hataları yakalanır, tasarım kusurları kaçar. Önce nedeni oku, sonra testlere bak, farkı yaklaşımı bilerek oku. Bu sırada on dakika, yukarıdan göz gezdiren bir saati yener; yazarlar nedeni önce yazarak kolaylaştırmalıdır.

Lab ve sonraki adım

L12 labı tarihe kurgusal gizli örneği yerleştirir; tespit, etki ve doğru müdahale sırasını değişiklik özeti olarak ister. Buradan yol M05'e sürer; sürümlü yapılar çalışabilir olanlara dönüşür: konteynerler.