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”
@Jarek wspomniałeś że wkurza Cię gdy podczas pisania kodu masz errory StyleCop owe. Polecam ten sposób: https://krzysztofmorcinek.wordpress.com/2017/12/13/stylecop-jakie-severity-warning-czy-error-dla-problemow-zglaszanych-przez-stylecop/
Krzysiek – dziękówka, sprzedałem temat w projekcie, mam nadzieję że coś się stanie w tym temacie.
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ę.