[Murasame]

Ten uczuć kiedy na daily pokazują ci, że żaden z twoich ukończonych tasków nie działa. Po czym sprawdzam historię zmian i junior zamiast pobrać moje zmiany i je zmergować, po prostu wstawił swoje i moje kompletnie wywalił xD

#programowanie #pracbaza

[80kg]

Zdarza się

[mlody_trabant]

@Murasame młody nie podumał i takie są efekty

[Murasame]

@80kg: @mlody_trabant

Ale to już nie pierwszy raz i dlatego mnie to irytuje.

[Pan_latarka]

@Murasame: Machający papież że można zmergować zmiany bez code review

[Murasame]

@Pan_latarka:

Niestety ale projekt pomimo, że robiony w bardzo dużym korpo, to bardziej przypomina klepanie kodu samemu w domu xd

Jest tak, że przychodzi np. jakiś stażysta na parę miesięcy i od razu wolno mu wrzucać swoje zmiany itd. nikt nie robi żadnego code review. Ja się nie wtrącam w to wszystko xD Potem mam co robić jak naprawiam po nich

[kakodyl]

@Murasame: To wygląda gorzej niż klepanie w domu. Takiego cyrku to nawet u Janusza nie widziałem.

Ale coś ci podpowiem, słyszałeś o czymś takim jak oddolna inicjatywa? Dość często udaje się wprowadzić zmiany w procesie które poprawiają nie tylko finalną jakość oprogramowania ale także szeroko pojęty DX (development experience)

[matt-pitt]

zamiast pobrać moje zmiany i je zmergować, po prostu wstawił swoje i moje kompletnie wywalił

@Murasame: Jakbyście mieli review w procesie i dobrze gita skonfigurowanego to coś takiego by nie miało miejsca ¯\(ツ)/¯

[Murasame]

@kakodyl:

To jest dość specyficzny projekt. Częściej to wyglądało tak, że sam sobie klepałem jakieś rzeczy w starych projektach i cały czas mam na sobie różne projekty, które sam ogarniam jak trzeba coś zmienić albo coś się wysypie. Teraz akurat mamy coś nowego i trochę większego, to pracują nad tym 4 osoby. I o ile byli to przedtem sami seniorzy/mid to w sumie wszystko bez problemu działało, bo tacy ludzie

[jar-mitryn]

@Murasame: to jak mergujecie zmiany? Wasz tool daje taką możłiwość, czy ręcznie kopiujecie? Macie coś na kształt branchy?

[eustach]

@Murasame: kakodyl dobrze ci napisał. Wiesz jak to powinno działać to weź wyjdź z inicjatywą i popraw te procesy. Wymuś robienie code review, mergowanie tylko po approvalu seniora itp. Tobie się będzie lepiej pracowało a może też ktoś wyżej to doceni i dadzą ci jakiś hajsik/awans. Olewanie tematu i mówienie że to nie twoje obowiązki jest słabe.

[Murasame]

@eustach:

Olewanie tematu i mówienie że to nie twoje obowiązki jest słabe.

Serio to już temat na inną dyskusję. Trochę poznałem to korpo, za nadmiar obowiązków nikt podwyżki nie daje. Jedyne co się liczy to dobra gadka i trochę szczęścia. Znam osoby, które cenię i smiało mogę uznać za o wiele mądrzejsze ode mnie a do tego mega przyklądają się do swojej pracy, często nawet za bardzo, poświęcając swoje zdrowie i czas. Każdy ich chwali a zarobki o tym nie świadczą xD Jednocześnie znam przypadki, gdzie akurat ktoś był potrzebny w danym momencie, rzucał papierami i nagle dostawał 100% więcej i bez znaczenia była cała reszta, zwykły fart. Z tego powodu ja już bardziej myślę o zmianie pracy niż o angażowaniu się tutaj bardziej.
Ale temat code review zamierzam poruszyć, bo przedtem to były rzadkie przypadki a coraz częściej biorą jakichś kompletnie

[cppguy]

@Murasame:

jakis #!$%@? junior ci sie trafil po prostu

[jar-mitryn]

@Murasame: znam podobne historie z autopsji, mieliśmy nieograniczonego SVNa. Też były cyrki. swoją droga myślałes o trzymaniu sobie lokalnie tego jako repo git, przynajmniej będziesz miał swoją historię zmian