[budyn]

#programowanie #programista15k #zalesie #procbaza #problemypierwszegoswiata

Mam taką jedną programistkę, która od 3 miesięcy walczy nad jednym (dosyć skomplikowanym) featurem.
Najgorsze już za nią. Daje PR kod do code-review.
Patrzę na szybko, kilkanaście komentarzy z zakomentowanym kodem, debug statements itd.
Dalej nie patrzyłem i piszę

Could you please go through your changes in full and try to do some clean up first? Thank you

Myślicie, że mnie wyrzucą za mobbing?

Tak się pracuję niestety w niektórych firmach na zachodzie (╯︵╰,)

#truestory

[Jake_921]

@budyn: W jakim kraju siedzisz?

[s.....e]

@budyn: żeby się za code review obrazić to już nie wiem kim trzeba być.

[budyn]

@Jake_921: Szkocja

[kywmn]

@budyn: nk zawoła jak się wyjaśni xD

[y.....l]

@budyn: Ja zawsze pisze, żeby wywalić wszystkie debugi i zakomentowany kod, a resztę po prostu sprawdzam.

[C.....y]

@budyn: wypunktuj komentarze w procesie review i tyle

[pokey]

@yggdrasil: mozecie wytlumaczyc co to jest „zakomentowany kod”?

[rotflolmaomgeez]

@pokey: kod, który nie jest aktywny, bo został oznaczony jako komentarz do kodu (komentarze są ignorowane). Ogólnie nie powinno się takich rzeczy zostawiać w kodzie, bo utrudniają czytanie.

[Nadrill]

@pokey: ekwiwalent folii ochronnej/naklejek producenta na agd. jesli wszystko jest gotowe to powinno sie je wywalic a nie zostawiac, bo przeszkadzaja i sugeruja ze nie wszystko jest gotowe do uzytku

[eth7]

@budyn Eh, a ja bym chciał żeby ktoś zrobił mi code review (╥﹏╥)

[tzim]

@eth7: w czym piszesz. Na pewno ktoś Ci z wypoku pomoże :D

[sh2dov]

@budyn: 15k GBP?

[jalop]

@sh2dov: raczej cebula wraca do Polski, ale 15k gbp całkowicie na miejscu.

[budyn]

@sh2dov: Pracuję zdalnie 90% czasu

[rorschach]

@budyn: jak nic molestowanie w pracy

[zwei]

@budyn: jak to się skończyło mirku

[budyn]

@zwei: Zwolniła się, mówiąc, że powodem jest powrót w rodzinne strony (była z Irlandii)

[SkorpionX]

@budyn niektóre firmy mają już politykę zero komentarzy w kodzie, nie ważne jakich i to się IMHO sprawdza. Czysty kod opisuje się sam, a jasno określone odgórnie zasady chronią osoby robiące CR przed tego typu wybuchami.

[a.....e]

@budyn: WTF