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.

2 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

Dodaj komentarz

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