Efektivní code review není o stylu kódu. Tady je na co se opravdu zaměřit.
Funkčnost¶
- ☐ Řeší PR skutečný problém?
- ☐ Edge cases ošetřeny
- ☐ Error handling kompletní
- ☐ Backward kompatibilita zachována
Čitelnost¶
- ☐ Kód je srozumitelný bez vysvětlování
- ☐ Naming jasný a konzistentní
- ☐ Komentáře tam, kde jsou potřeba (ne všude)
- ☐ Funkce/metody krátké a jednoúčelové
Bezpečnost¶
- ☐ Žádné hardcoded secrets
- ☐ Input validace
- ☐ SQL injection prevence
- ☐ XSS prevence
Performance¶
- ☐ Žádné N+1 queries
- ☐ Vhodné indexy pro nové queries
- ☐ Caching kde má smysl
- ☐ Neblokující operace pro I/O
Testy¶
- ☐ Unit testy pro novou logiku
- ☐ Integration testy pro nové endpointy
- ☐ Edge case testy
- ☐ Test coverage neklesá
Ostatní¶
- ☐ Dokumentace aktualizována
- ☐ Migrace reversibilní
- ☐ Feature flag pro rizikové změny
- ☐ Changelog entry
Kultura¶
Code review je konverzace, ne výslech. Buďte konstruktivní, navrhujte řešení, ptejte se „proč”.
code reviewbest practiceskvalita