aula polska - psychologia i code review
TRANSCRIPT
Psychologia i Code ReviewTomasz Grzemski (macopedia.co)
Tomasz Grzemski (macopedia.co)
Macopedia.co - Founder
Enterprise Devops
Aviation geek
Psychologia i Code Review
Magento
Psychologia i Code Review
Macopedia
Offices
3
Psychologia i Code Review
Developers
15+
Magento Certified Devs
3+
Klienci
50+
Rynki
10+
Magento
Psychologia i Code Review
Rozwiązania istniejącego problemu
Sposobu na zwiększenie przychodów
Sposobu na optymalizację kosztów
Czego nasi klienci poszukują?
Magento
Psychologia i Code Review
Dostarczenie nowego narzędzia biznesowego!
Psychologia i Code Review
1. Requirments analysis2. Backlog Plannaing
3. Sprint planning4. Daily meetings
5. Sprint demo6. Retrospective
2 weekscycle
Nasz proces zarządzania
Psychologia i Code Review
Nasz proces wytwarzania
Code Quality testAcceptance tests
CountinousIntegration
Cloud Deployment
Documentation Training
Local Development
Git Repository
Magento
Psychologia i Code Review
Czego klienci nie poszukują?
Efekt oczekiwany różni się od efektu końcowego
Braku kompetencji
Błędów / niskiej jakości produktu
Devs
PM
}
Magento
Psychologia i Code Review
Jak przeciwdziałać?
Testy automatyczne
Testy ręczne
Szkolenia
Motywacja
Wymiana wiedzy
Code Review
Magento
Psychologia i Code Review
Jak przeciwdziałać?
Testy automatyczne
Testy ręczne
Szkolenia
Motywacja
Wymiana wiedzy
Code Review
Psychologia i Code Review
Code review / Przegląd kodu
Psychologia i Code Review
Przegląd kodu?
Psychologia i Code Review
Jaki cel?
Co można zrobić lepiej?
Czy zadanie jest zrobione zgodnie z wymaganiami?
Czy nie ma potencjalnych luk w bezpieczeństwie?
Czy „zamiatamy pod dywan” czy rozwiązujemy problem?
Psychologia i Code Review
Jak?
Programista
Oceniający
Oceniający
Dobra robota!
Dobrze, ale popraw
to!
Psychologia i Code Review
Kiedy?
Psychologia i Code Review
Przegląd kodu?
Komunikacja
Psychologia i Code Review
Psychologia
Psychologia i Code Review
Kto ma prawo oceniać?
(szef)
(programista)
Psychologia i Code Review
Kto ma prawo oceniać?
Efekt:
- brak pracy zespołowej
- autorytarność
- brak wymiany wiedzy
- obawy / brak komfortu
(szef)
(programista)
Psychologia i Code Review
Kto ma prawo oceniać?
(szef)
(programista)
Psychologia i Code Review
Kto ma prawo oceniać?
Zespół!
Psychologia i Code Review
Kto ma prawo oceniać?
Niekończąca się dyskusja!
Psychologia i Code Review
Akceptujemy?
Demokracja!
+1-1
+1
Źródło: http://sback.it/publications/icse2013.pdf
Psychologia i Code Review
Psychologia i Code Review
Haters gonna hate!
„Jak on może oceniać moją wybitną pracę”
„On ma mniejszą wiedzę i wymądrza się”
„Nie lubię go personalnie i jeszcze wytyka moje błędy”
Psychologia i Code Review
Brak akceptacji?
Nie lubisz code review - jak chcesz pracować w zespole?
Nie lubisz być oceniany - jak chcesz polepszyć swoje umiejętności?
Popełniasz nieświadomie błędy - w jaki sposób chcesz się o nich dowiedzieć?
Psychologia i Code Review
Podstawa Code-Review!
Motywacja zespołu!
Psychologia i Code Review
Proste zasady komunikacji!
Oceniaj z szacunkiem - tak jak byś ty chciał być oceniony
Podpowiadaj rozwiązania - nie wyśmiewaj to co zostało zrobione
Jeśli masz lepszy pomysł - uargumentuj to!
Zauważysz brak kompetencji - zaproponuj przeprowadzenie szkolenia!