[K.....z]

Czy to normalne mieć code review poza zespołem? ( ͡° ͜ʖ ͡°)

#programowanie #pracait

[kam3o]

@KorczakSzcz: o ile ktoś z innego zespołu jest bardziej kompetentny w danym temacie to czemu by nie

[Zelber]

@KorczakSzcz: Nie da się odpowiedzieć na to pytanie tak/nie, bo odpowiedzią jest "to zależy"

[groman43]

@KorczakSzcz: Pytanie, czy osoba robiąca to review, wie o co kaman - na czym polega ficzur, jak działa system, jakie są wymagania funkcjonalne oraz niefunkcjonalne, etc. Jeśli nie, to takie review za dużo nie jest warte.

[K.....z]

@kam3o: @groman43: @Zelber: zna domenę i ogarnia aplikacje

[groman43]

@KorczakSzcz: No to nie widzę problemu.

[D.....a]

@KorczakSzcz: Brakuje odpowiedzi "co mieć?" :D

[Krolik]

@groman43: kod to tylko kod, brak kontekstu to dobra rzecz, bo od razu sprawdzasz czy kod jest napisany zrozumiale. IMHO jedno review powinno być robione przez osobę z zewnątrz. To osoby wewnątrz zespołu często robią słabe review, bo nie zauważają oczywistych problemów z czytelnoscia i brakiem dokumentacji - bo dla nich wiele rzeczy jest oczywiste.

[groman43]

@Krolik: Bez znajomości kontekstu nie jesteś w stanie sprawdzić, czy zaimplementowany ficzur działa tak jak powinien. Poza tym, nie możesz sprawdzić, czy kod nie łamie założeń architektonicznych, ani nie psuje innych, już wcześniej zaimplentowanych ficzurów. Poza tym, bez wiedzy domenowej często zrozumienie kodu, jest po prostu niemożliwe.

[awiner]

@groman43: od sprawdzania czy działa jak powinien są QA a nie code review

[kimikini]

@KorczakSzcz: no to git jak tak

[groman43]

@awiner: Im szybciej jest wyłapany błąd, tym później łatwiej i taniej go naprawić. Nie wyobrażam podczas review nie sprawdzać, czy ficzur został zaimplementowany poprawnie.