Ten w którym rozmawiamy o code review

Przedstawiamy wam 18 odcinek podcastu Ostra Piła w którym to rozwodzimy się nad code-review. Porozmawiamy sobie na takie tematy jak to jak robić dobrze code review oraz czy każdy powinien je robić czy może tylko seniorzy powinni dostąpić tego zaszczytu? Dwugodzinne code-review dla waszej programistycznej przyjemności. Miłego słuchania.

Notatki:

– nowe miejsce do słuchania https://www.podchaser.com/podcasts/podcast-ostrapila-511648

– https://twitter.com/iamdevloper/status/397664295875805184

– long-lived vs short-lived DbContext http://mehdi.me/ambient-dbcontext-in-ef6/

– http://www.fitandforty.org/thats-the-way-things-have-always-been-done-around-here/

– http://www.radiowawa.pl/detektyw-sledztwa

– https://www.youtube.com/watch?v=Gz-DG-ys_f0

– https://mtlynch.io/human-code-reviews-1/

– https://mtlynch.io/human-code-reviews-2/

– builders prank: https://www.youtube.com/watch?v=v-ZZ3Qc0iF8

– punkty za podpierdalanie: https://www.wykop.pl/wpis/13817475/

 

Jeśli się podobało – daj plusa, lajka lub komentarz. Dzięki i do następnego razu.

4 odpowiedzi do “Ten w którym rozmawiamy o code review”

  1. Garść komentarzy z mojej strony:
    – Do literówek w kodzie są dodatki (Spell Checker, Code Spell Checker, ReSpeller)
    – StyleCop a spacje i nawiasy – powinno być suggestion, nie dziwię się że jest frustracja, ale wiadomo to decyzja całego zespołu
    – Formatowania kodu, gruby refactor (rename, move) – oddzielny commit/pull request
    – Skarbonka na 2 zł to dobry sposób xD
    – „Zamień na Linq” – należy uważać, czasem zmniejsza czytelność kodu
    – Pozytywne komentarze w code review to bardzo dobra rzecz!
    – Automatyzacja znajdowania literówek jest możliwa – https://tpodolak.com/blog/2017/10/27/finding-typos-resharper-command-line-tools-respeller-cake/
    – Zdecydowanie jestem zwolennikiem przeglądania wszystkich zmienionych plików przed commitem
    – Stop tekstom typu ble ble, to mało profesjonalne. 2 min. nikogo nie zbawi żeby wymyśleć adekwatna nazwę/frazę/tekst

  2. Łukasz – dzięki za materiały, będę patrzeć.
    Co do ‚ble ble’ – to trochę się nie zrozumieliśmy. Nie chodziło mi o aby wpisywać bleble, tylko że ten teksty nie muszą być „pixel” perfect, dopuszczam literówki, czy inną drobnicę.

Dodaj komentarz

Twój adres email nie zostanie opublikowany. Pola, których wypełnienie jest wymagane, są oznaczone symbolem *