[ChamskoCytuje]

Zaraz zaczynam nową robotę i pierwszy raz trafiam do firmy gdzie jest code review ( ͡° ͜ʖ ͡°) Jak rozumie to jest taki roast dla programistów? Mam nadzieje ze nie beda mnie tam bardzo jechali, bo planuje tam dłużej popracować ( ͡° ͜ʖ ͡°)
#programowanie #pracbaza

[JakTamCoTam]

@ChamskoCytuje: to że jest formalnie code review to jeszcze nic nie znaczy ( ͡° ͜ʖ ͡°) w mojej poprzedniej firmie to ograniczał się do plucia na nadmiarowe spacje xD

[ChamskoCytuje]

@JakTamCoTam: no wlasnie nie wiem na co tu beda zwracac uwage i czy w ogole komus bedzie sie chcialo każdego mojego commita przegladac.

[JakTamCoTam]

@ChamskoCytuje: no tak, to norma. Ty też będziesz musial

[m.....s]

@ChamskoCytuje: korzystaj ile wlezie z code review. To nie jest czepialstwo, tylko szansa na naukę po obu stronach. Przykładowo - Twój kod może i będzie działać z uzgodnioną specyfikacją i wymaganiami biznesowymi, ale jak biznes wyskoczy z jakimiś zmianami to powinien być łatwo rozszerzalny. Jeżeli nie jest - code review powinien to wychwycić.

[groman43]

@ChamskoCytuje: Nie, code review to nie roast. Code review ma zwykle na celu dwie rzeczy:
- wyłapanie wszystkich potencjalnych problemów
- zaznajomienie się innych członków zespołu z Twoimi zmianami
W przypadku juniorów code review dodatkowo pomaga ocenić progres oraz nakierować na właściwą drogę ( ͡° ͜ʖ ͡°)
Code review to nie okazja do roastowania czy obrażania innych. Jeśli ktoś nie zgadza się z Twoimi zmianami powinnien wyjaśnić

[Passer93]

@ChamskoCytuje: code review jest konieczny, sam odrzuciłem jedna ofertę pracy bo nie mieli obowiązkowego code review

[stackhouse]

@ChamskoCytuje: najpierw masz wrażenie, że się wszystkiego czepiają na code review a potem odkrywasz taką ciekawą książkę jak clean code i nagle się okazuje, że to wszystko ma sens ( ͡° ͜ʖ ͡°)

[keton22]

@ChamskoCytuje: code review poza tym co napisali Mircy powyżej to też potężny dupochron jak naknocisz to nie jest tylko twoja wina ale głównie tego co nie wyłapał bo on jest 'starszy i mądrzejszy'.

[m.....s]

@ChamskoCytuje: a, jeszcze dodam, że jeśli ktoś robiący code review traktuje ten proces jako okazję do podreperowania ego i pokazania kto tu rządzi to zdecydowanie minął się z pracą którą wykonuje bo z takim podejściem po prostu obrzydzi ten proces osobie sprawdzanej.

Każdy popełnia błędy, niezależnie od poziomu stanowiska jakie zajmuje oraz rzecz jasna wiedzy i doświadczenia. Sam nawet leadowi czy seniorowi znajdywałem rzeczy, które można było zapisać inaczej np. aby

[printHelloWorld]

@ChamskoCytuje: praca na jakim stanowisku jak można wiedzieć?

[wszystkieInneBylyZajete]

@ChamskoCytuje: