Dobré review se ptá, co se změnilo v chování, kontraktech a provozu. Nestačí, že nové řádky vypadají úhledně.
Diff je mapa, ne území
Nástroj na review ukazuje přidané a odebrané řádky. Uživatelé ale zažívají chování. Než začnete připomínkovat syntaxi, pojmenujte kontrakt, do kterého změna zasahuje: API, pravidlo ukládání, pracovní postup provozu nebo očekávání jiného týmu.
Přečtěte popis změny, podle možností vyzkoušejte důležitou cestu a podívejte se do okolního kódu. Malý diff ve sdílené části systému může nést větší riziko než rozsáhlá izolovaná změna.
Na hranicích chtějte důkaz
Testy posuzujte proti zamýšlenému chování, včetně chybových a krajních stavů. Co se stane při chybějících datech? Se starším klientem? Při opakování po částečném úspěchu? Pokud změna obsahuje migraci, mohou stará a nová verze bezpečně fungovat současně?
Pište připomínky, které popisují konkrétní riziko a způsob jeho ověření. „Tohle se mi nezdá“ se řeší těžko. „Opakovaný požadavek může vytvořit dvě faktury, protože se mění klíč“ dává autorovi testovatelný problém.
Ať je review užitečné i lidské
Oddělujte rizika, která blokují vydání, od osobních preferencí. Vysvětlete důvod požadavku a pojmenujte kompromis. Opakující se mechanické připomínky patří do týmového stylového návodu, aby lidská pozornost zůstala pro chování a návrh.
Cílem je společné porozumění změně. Dobré review pomůže autorovi i čtenáři udržovat kód ještě za půl roku.