Code review

Každý merge request pred zlúčením skontroluje druhý člen tímu. Review nie je hľadanie vinníka – je to najlacnejší moment, keď sa dá chyba nájsť, a zároveň sa pri ňom naučíte, ako rieši veci kolega.

Ako na to

1. Prečítajte si plán úlohy (v popise merge requestu) a akceptačné kritériá user story.
2. Stiahnite si vetvu a spustite appku – nestačí pozrieť len kód.
3. Prejdite kód v GitLabe a k problémom pridajte komentár priamo na riadok.
4. Keď je všetko v poriadku, merge request schváľte. Keď nie, napíšte, čo treba opraviť, a po oprave sa pozrite znova.

Checklist

• Appka funguje a spĺňa všetky akceptačné kritériá user story – vyskúšajte ich v appke.
• Sú vyriešené aj prázdny stav, načítavanie a chyba, ak ich user story má.
• Kód dodržiava technické zásady – štruktúru, pomenovanie, farby z palety, komponenty.
• Nič sa neopakuje – rovnaký kus šablóny alebo logiky nie je skopírovaný na viac miestach.
• V kóde nezostali console.log, zakomentovaný kód ani nepoužívané súbory.
• V konzole prehliadača nie sú chyby.
• Merge request obsahuje len zmeny k tejto úlohe.

Ako písať komentáre

• Píšte o kóde, nie o človeku: „Táto funkcia sa volá dvakrát“, nie „Zase si to zle spravil“.
• Buďte konkrétni: napíšte, čo je zle a prečo, a ak viete, navrhnite riešenie.
• Rozlišujte, čo je nutné a čo len návrh. Návrh začnite slovom „Návrh:“ – autor sa môže rozhodnúť, či ho prijme.
• Pýtajte sa, keď niečomu nerozumiete. Aj otázka je dobrý komentár.
• Pochváľte, čo sa podarilo.

Kedy schváliť

Merge request schváľte, keď je celý checklist splnený a všetky nutné komentáre sú vyriešené. Návrhy nemusia byť zapracované. Ak sa s autorom neviete dohodnúť, opýtajte sa mentora.