Praktyczne Code Reviews

Sebastian Marek (25.Oct.2013 at 17:15, 1 hr )
Talk at PHPCon Poland 2013 (Polish)

Rating: 5 of 5

Wbrew powszechnym opiniom, nie tak prosto jest zrobić dobre Code Review. Robione w pośpiechu, tylko po to by je "odbębnić", często stwarza więcej szkody niż pożytku. Opowiem wam dlaczego code review jest ważne i jak wykorzystać ten proces aby upewnić się, że napisany kod jest najwyższej jakości. Będę nie tylko mówił o tym kto powinien robić code reviews, i dla kogo, jakie informacje są potrzebne do przeprowadzenia skutecznego code review, ale także jak zrobić dobre code review w najkrótszym możliwym czasie.

Who are you?

Claim talk

By clicking this button you are declaring that you are the speaker responsible for it and a claim request will be sent to the administrator of the event.

If the claim is approved you will be able to edit the information for this talk.

Are you sure?

 
Comments closed.

Comments

Rating: 5 of 5

25.Oct.2013 at 17:59 by Dawid Polak (1 comments)

Świetne slajdy. I naprawdę ciekawa dynamiczna prezentacja dobra na początek #phpcon

Rating: 5 of 5

25.Oct.2013 at 18:06 by Piotr Brzezina (1 comments)

Bardzo ciekawy wykład. Spory nacisk na sposoby komunikacji w zespole.

Rating: 5 of 5

25.Oct.2013 at 18:15 by Nikodem O. (10 comments)


Rating: 4 of 5

25.Oct.2013 at 18:27 by Krzysztof Menżyk (24 comments)

Ciekawy wykład. Brakowało jeszcze pokazania pełnego flow przeprowadzenia code review w przykladowym narzędziu.

W kontekście narzędzi można również wspomnieć o CS Fixerze ( https://github.com/fabpot/PHP-CS-Fixer).

Jestem ciekaw również jak używasz sonara w swoich projektach.

Rating: 3 of 5

26.Oct.2013 at 10:42 by Anonymous


Rating: 5 of 5

26.Oct.2013 at 11:45 by Anonymous

Jak do tej pory najlepsza rezentacja

Rating: 4 of 5

26.Oct.2013 at 12:00 by Anonymous

fajna i przydatna prezentacja, ale chciałbym usłyszeć ciut więcej porad jak prowadzić code review

Rating: 5 of 5

26.Oct.2013 at 15:27 by Piotr Krzysztofik (2 comments)

Bardzo ciekawy temat i super przedstawione. Wiecej takich wykladow ;-)

Rating: 5 of 5

26.Oct.2013 at 18:05 by thejw23 (13 comments)

Bardzo fajna prelekcja, zarówno treść jak i forma, więcej takich :)

Rating: 4 of 5

27.Oct.2013 at 08:01 by Anonymous

Ciekawa prezentacja. Czego mi brakowalo? Jedna z zasad code review jest to zeby utrwalic wyniki dla zespolu. Chcialbym zobaczyc na koniec kilka takich realnych przykladow, ktore oprocz narzedzi pokazaly by tresc recenzji.

Rating: 4 of 5

27.Oct.2013 at 08:01 by Anonymous

Przydatna, prawdopodobnie zacznę korzystać z sonara.

Rating: 4 of 5

27.Oct.2013 at 08:36 by Dominik Czajka (5 comments)

Prezentacja spoko, choć nic nowego jak dla mnie. Przydatne dla zespołów nie stosujących code review.

Rating: 5 of 5

27.Oct.2013 at 10:05 by Krzysztof Szala (5 comments)

Doskonale poprowadzona prelekcje. Prowadzący z dużą wiedzą i zdolnościami oratorskimi. Jedna z lepszych prelekcji na tej konferencji.

Rating: 5 of 5

27.Oct.2013 at 12:35 by Anonymous

Zdecydowanie najlepsza prezentacja. Czuć było, że autor jest doświadczony i zna dobrze temat. Dzięki!

Rating: 4 of 5

27.Oct.2013 at 17:30 by Wojtek Iskra (18 comments)


Rating: 5 of 5

27.Oct.2013 at 18:02 by Anonymous

Świetna prezentacja - dużo ważnych dla mnie spostrzeżeń. Dzięki.

Rating: 5 of 5

27.Oct.2013 at 18:32 by Wojciech Sznapka (11 comments)

Pomimo, że widziałem po raz drugi, to ciągle interesował mnie temat i doświadczenia przekazywane audiencji. Bardzo dobry sposób prezentacji no i ładne slajdy. Congrats!

Rating: 5 of 5

27.Oct.2013 at 19:24 by Tomasz Wasiński (12 comments)

Świetnie poprowadzone, przedstawione wszystko jak trzeba, jak dla mnie jedna z lepszych :) Dzięki !

Rating: 5 of 5

27.Oct.2013 at 20:49 by Anonymous

Dobrze przygotowania prezentacja na ważny dla developerów temat. Co prawda niewiele z niej wyniosłem, ale utrwaliłem sobie wiedzę.

Rating: 5 of 5

28.Oct.2013 at 08:56 by Vokiel (33 comments)

++ za formę i treść

Rating: 5 of 5

28.Oct.2013 at 09:10 by Radek Benkel (40 comments)

Mamy w środowisku PHP trochę ”freaków” – w pozytywnym znaczeniu tego słowa, jak i negatywnym ;) Jednakże, Sebastian Marek ma pozytywnego bzika punkcie jakości kodu i automatyzacji. I to od razu widać.

Prezentacja m.in. o tym, jak robić Code Reviews, aby miały sens – czyli „kolego z biurka obok, popchnij mi to dalej bo release za 3 minuty” to nie jest to, o co chodzi code reviews. Posłuchaliśmy także o komunikacji w zespole. Trochę o narzędziach – czego warto używać, czego nie używać (email to nie jest dobry sposób na CR) oraz sporo o procesie. Wyniosłem z prezentacji kilka przydatnych rad.

Uwaga co do slajdów – kiedy już tłumaczone są na polski (a z tego co widzę, to tak chyba było), to odpuśćmy sobie tłumaczenie takich słów jak „branch” – słowo „gałąź” na prezentacjach technicznych źle się czyta ;)

Rating: 3 of 5

28.Oct.2013 at 09:46 by Anonymous

Prezentacja ogólnie na duży plus ale nie podobały mi się niektóre zdania np: że zbieranie statystyk z liczby commitów czy add line mogą posłużyć Twojemu szefowi jako wyznacznik pracy :(.

Programowanie to nie praca na akord wymaga motywacji a czasem i odpoczynku 3 dni, żeby napisać 10 linii, które zmienią świat :)

nie ilość a jakość!

Rating: 5 of 5

28.Oct.2013 at 10:24 by damian Rawski (13 comments)

Bardzo charyzmatyczny prelegent - widać, że wszystko co mówił, wynikało bezpośrednio z jego doświadczeń. Zachęcił mnie do znacznie bardziej sumiennie przeprowadzanych code review!

Rating: 5 of 5

28.Oct.2013 at 10:58 by zordon (4 comments)

Świetna prezentacja, dla mnie nr 2 całej konferencji. Dobre slajdy, charyzma, duża wiedza poparta doświadczeniem, a najważniejsze - umiejętność przystępnego jej przekazania

Rating: 3 of 5

28.Oct.2013 at 11:18 by Anonymous

Fajnie poprowadzona prezentacja. Trochę jednak zabrakło konkretów. Dla osób, które znają temat nic nowego - np. na co zwracać uwagę przy CR, jak robić CR, czy manualne przetestowanie obszaru commita powinno być częścią CR czy już nie. Stwierdzenie, że CR nie powinno służyć jako zastępnik checkstyle to trochę za mało. Ale ogólnie ok.

Rating: 5 of 5

28.Oct.2013 at 12:27 by Mariusz Gil (6 comments)

Miło się słucha kogoś, to ma doświadczenie w temacie i potrafi się podzielić swoją wiedzą. Dodatkowy plus za propagowanie idei CodeClub i zainteresowanie tym tematem paru osób na sali.

Rating: 5 of 5

28.Oct.2013 at 15:29 by Mateusz Kamiński (20 comments)


Rating: 5 of 5

28.Oct.2013 at 19:25 by Sergii Pryz (10 comments)

Thank you very much!
Code review should be a part of project workflow. Nobody will argue with that.
But the only one question: what is a best way to do that? Thank's Sebastian we know how: the first thing is a respect to the reviewer, the second use automatization tool before deep look to the code, the last review the "problem" parts.

Rating: 5 of 5

28.Oct.2013 at 19:50 by Anonymous

Zdecydowanie jedna z lepszych na konferencji. Nic dodać, nic ująć.

Rating: 4 of 5

29.Oct.2013 at 19:42 by Wojciech Muła (11 comments)

+ Podstawowa wiedza przedstawiona w bardzo przejrzysty sposób. Dla osób, które CR nie mają lub zaczynają, to z pewnością było ok.

- Z punktu osoby, która CR praktykuje, brakowało niekiedy konkretów, jakiegoś przykładu z życia, może nawet pełnego przejścia takiego procesu.
- Zabrakło czasu na pytania z sali, co jak pokazały inne wykłady, może dodać sporo do tematu.

Rating: 5 of 5

29.Oct.2013 at 21:06 by Przemyslaw Koltermann (13 comments)


Rating: 5 of 5

29.Oct.2013 at 22:19 by Piotr Dutko (8 comments)

Świetna prezentacja. Fajnie przedstawiona. i temat ciekawy :)

© Joind.in 2014