[Kresse]

Tez macie takich ananasow, ktorym na code review ladnie zaargumentujecie czemu ich rozwiazanie ich zle, zaproponujecie alternatywe, a jako reakcje dostajecie jedynie komentarz "Ok, but I like my approach more"? xD Zawsze staram sie byc mily, ale korci mnie, zeby troche go pocisnac (ಠ‸ಠ)

#programowanie

[whoru]

@Kresse: ja takich #!$%@? do leadów od razu

nie rozumiem czemu koledzy z pracy mnie nie lubią

[sgt_colbert]

@Kresse: czemu rozwiązanie jest złe?

[Aryze]

@Kresse: To nie ananas, tylko debil ( ͡° ͜ʖ ͡°)

[Kresse]

@sgt_colbert: Bo jest slabo testowalne, slabo czytelne, dodaje ignora do lintera w miejscach gdzie nie jest to konieczne i zawiera konstukcje, ktore w przyszlosci moga byc zrodlem bledow (np. deklaruje funkcje, ktora wywoluje dowolna lambde przekazana jako parametr jako funkcje czysta).

[sgt_colbert]

@Kresse: w takim wypadku jak Ci zależy na jakości to bym przegadał to w obecności osoby kierującej projektem, ale z doświadczenia wiem, że to droga przez mękę. Wg mnie jedyne rozwiązanie to dedykowany tech lead który ma autorytet i przyzwolenie na odrzucanie takich PR o ile pozwala na to sytuacja w projekcie. Jak ktoś jest oporny to w sytuacji dev vs dev zwyczajnie już to zlewam, a jak stan kodu spowalnia

[sgt_colbert]

@Kresse: a pytam bo czasem niektórzy przeginają w drugą stronę i na review forsują swoje jedyne słuszne rozwiązanie ( ͡° ͜ʖ ͡°) ale jak kod łamie podstawowe praktyki to dobry argument

[Kresse]

@sgt_colbert: No ja w sumie mam troche leadowa pozycje, ale nie lubie narzucac swojego rozwizania "bo tak", tylko wymieniam problemy jakie stwarza dany kod i wiekszosc dev'ow to rozumie i poprawia, albo ewentualnie znajdujemy jakis kompromis, jesli dana rzecz faktycznie "zalezy tylko od gustu". Tylko ten jeden dev jest jakis uparty i chce wywracac caly projekt do gory nogami xD

[rimyi]

@Kresse: To nie akceptuj mu PR, proste

[IronHeisenberg1939]

@Kresse: Miałem takiego ananasa kiedyś w pracy, na jego korzyść przemawiało to że miał więcej doświadczenia i był dłużej w firmie, też tak reagował na moje rozwiązania, dużo kodu powielał i nie dało się mu wytłumaczyć.
Najlepsze w tym było to że przez jego błędy aplikacja dostawała negatywy na sklepie a często jego kod powodował to że trzeba było poświęcić kilka tygodni na refaktor, o testowaniu tego kodu nawet nie wspomnę