[NNPlaya]

#programowanie jak asertywnie komuś wytłumaczyć w code review, że kod który naprodukował nie nadaje się do wejścia dalej? Ziomuś natrzepał 3k linii kodu, sporo powtórzeń i zduplikował logikę walidacji w mikroserwisie, która istnieje gdzieś indziej a tu tylko przekazujemy dane żeby przeszły dalej :( a ten mi sprawdza, czy przypadkiem data końca rezerwacji nie jest wcześniej niż data początku. Musi być, bo inaczej nie udałoby się zrobić tej rezerwacji

[K.....s]

@NNPlaya: https://google.github.io/eng-practices/review/reviewer/

[MikelThief]

@NNPlaya: napisz mu to co tutaj jak lepiej nie umiesz ( ͡° ͜ʖ ͡°)

[aseeon_]

@NNPlaya: Co ma do tego wszystkiego asertywność? Piszesz mu co konkretnie jest nie tak i co powinien twoim zdaniem poprawić - to jest aż tak proste.

[g.....n]

@NNPlaya: może on nie wie że dane są już zwalidowane? Albo wie ale jest bardzo ostrożny, albo wcale nie są - albo są ale tylko po stronie klienta?

[NNPlaya]

@Kormas Dzięki, przydatne!
@aseeon_ przepraszam, zrobiłem skrót myślowy. Chodzi o to, że spodziewałem się oporu wobec moich uwag. Nie każdy dev przyjmuje lekko sugestie w swoim code review. Rzeczowa dyskusja jest ok, opór za wszelką cenę "bo już zrobione" nie bardzo. To o co naprawdę chciałem zapytać, to jak sobie radzić w takiej sytuacji. Na pewno pomocny okazał się link od @Kormas. Szczególnie sekcja "handling pushback"

[Z.....n]

@NNPlaya:

a ten mi sprawdza, czy przypadkiem data końca rezerwacji nie jest wcześniej niż data początku. Musi być, bo inaczej nie udałoby się zrobić tej rezerwacji

ja to bym raczej zapytał

z jakimi systemami musiał mieć do czynienia :D