Code Review

Code Review – założe ia

• Syste aty z ość

• Każda, a et d o a z ia a jest sp a dza a i usi p zejść ak epta ję

• Sp a dza y kod pod zględe jakoś i i zytel oś i odi g standards)

• Szuka y pote jal y h łędó o az pól do opty aliza ji

• Udzielamy porad i wskazówek!

• Code Review != Testowanie

to Code Review or not to Code Review

• Pod osze ie zytel oś i i jakoś i t o zo ego kodu

• Wy h yty a ie łędó o az pote jal y h luk logi e

• Pierwszy krok w znajdowaniu potencjalnych problemów z

ydaj oś ią aplika ji

• ) iększa y iedzę i u iejęt oś i zło kó zespołu

to Code Review or not to Code Review

• Duża zaso hło ość a y sp a dzić li ijek kodu pot ze a około 1h)

• Pot ze a duża iedza i dokład ość

• Spada szy kość ealizo a ia zadań

• Jakość i dokład ość Code Re ie zależy od oso y sp a dzają ej kod

• Współp a o i y ogą Cię z ie a idzić :

Pięć po ad

1. M iejsze z ia y łat iej i szy iej p zea alizo ać

2. Na Code Re ie p zez a za y iej iż h dzie ie

3. Code Review jest asynchroniczne, robimy je gdy mamy czas

4. Pozyty e asta ie ie! O e ia y kod a ie p og a istę

5. Code Re ie po i e yko y ać ały zespół

Pięć po ad

1. M iejsze z ia y łat iej i szy iej p zea alizo ać

2. Na Code Re ie p zez a za y iej iż h dzie ie

3. Code Review jest asynchroniczne, robimy je gdy mamy czas

4. Pozyty e asta ie ie! O e ia y kod a ie p og a istę

5. Code Re ie po i e yko y ać ały zespół

P o es któ y p o adziliś y

1. Pie sze Code Re ie o i p og a ista po zakoń ze iu zada ia

2. Następ ie t o zy y Me ge Re uest i z ie ia y status zada ia na `To Code Review`

3. Robimy Code Review, dodajemy komentarze i porady

4. Programista poprawia kod i zmienia status zadania na `To Code

Review` (o ile potrzebna jest poprawa)

5. K oki i po ta za y aż do uzyska ia satysfak jo ują ego kodu

6. Po akceptacji merge'ujemy branch i zadanie przechodzi na kolejne

etapy

Do Code Re ie uży a y GitLab

Do Code Re ie uży a y GitLab

Do Code Re ie uży a y GitLab

Na o z a a y sz zegól ą u agę

• Coding standards

• Opisy funkcji i klas

• Pop a a o sługa yjątkó

• Logowanie stanu aplikacji w kluczowych miejscach

• Przestrzeganie wzorca MVC i struktury Magento

• Pote jal e łędy logi e

• Zapytania SQL

Po t ze h iesią a h:

• 150 Merge Request'ów

• Około p o le ó ydaj oś io y h zostało z alezio y h podczas Code Review

• 9 % spadek p zypadkó złego stylu kodu

• 99.9% naszych metod i klas posiada docblock

• Nasz kod jest lepszy i bardziej czytelny

• Codzie ie u zy y się od sie ie

Piotr Studnicki info@divante.pl

ideas2action.pl

Dziękuję!