Kategorie
Podcast

Ten w którym rozmawiamy o code review – OstraPiła #18

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 na “Ten w którym rozmawiamy o code review – OstraPiła #18”

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

Ł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ę.

Możliwość komentowania została wyłączona.