[mk321]

#programowanie #pracait

Jeśli w zespole jest przykładowo kilku programistów i jeden architekt, to kto powinien robić code review?

Architekt nie ma czasu, żeby zająć się wszystkimi commitami.

Najlepiej by było jakby programiści robili sobie review wzajemnie. Ale jak nikt nie robi, bo każdy zajmuje się swoimi zadaniami? Jak powinna zapaść decyzja kto konkretnie ma zrobić review commita, który już długo leży?

- Kierownik ma się bawić takimi pierdołami (tzn. zajmować się wyznaczaniem osoby do każdego commita)?
- Programista, który zrobił commita, szukać kogoś, żeby go prosić kogoś, żeby mu zrobił review? A co jak nikt nie będzie chciał?
- Losowanie i na tego co wypadnie to musi zrobić?

[P.....k]

@mk321: frajer

[pamietacieczyjuzzapomnieliscie]

@mk321: co architekt robi w branży IT? ( ͡º ͜ʖ͡º)

[szary-obywatel]

@mk321U mnie w pracy to jest tak zrobione, że CRki mają pierwszeństwo. Jak zrobiłeś zadanie, PR i jest gotowe do sprawdzenia, to bierzesz CRke. Każdy sobie sam wybiera co ma sprawdzić. Ważne żeby było to robione na bieżąco

[mk321]

frajer

@Primusek: co?

[koostosh]

@mk321: programiści między sobą, powinien wybierać osobę która mu zrobi review bez pytania ale ze zdrowym rozsądkiem

[pamietacieczyjuzzapomnieliscie]

@mk321: i co to są commity, review? ( ͡° ʖ̯ ͡°)

[laki1]

@mk321: Ile macie lat xD

Prosta zasada jak jest jakiś pull/merge request bez review to nie możesz wziąć następnego taska

[P.....k]

@mk321: pytasz się kto powinien. Ja odpowiadam: frajer. ( ͡° ͜ʖ ͡°)

[s3b4]

@mk321: code review? a na co to, a komu to potrzebne ( ͡° ͜ʖ ͡°)

[mk321]

@szary-obywatel: @koostosh: @laki1: dzięki za odpowiedzi. Logiczne. Jak widać są różne podejścia w różnych firmach.

Jeszcze jedno pytanie: jak raportujecie czas pracy, to code review na co wpisujecie?
- Dołączacie do czasu swojego zadania, które ostatnio robiliście (i każdy tak robi, wiec sumarycznie wychodzi na równo)?
- Dołączacie do czasu zadania, którego robicie code review (czyli dodatkowy czas do zadania/projektu osoby, która napisała kod)?
- Oddzielne

[koostosh]

@mk321: nie raportujemy czasu pracy per zadanie. Programista ma 6 godzin programować, pozostałe 2 godziny dziennie ma na m.in. robienie rewiew

[laki1]

@mk321: Jak kiedyś musiałem raportować czas pracy to pisałem 'Code review - task #213' więc raczej pkt2

[PaulPavello]

@mk321: Review zazwyczaj praktykuje sie miedzy programistami :D Jeden pisze drugi sprawdza. Ja jestem w projekcie gdzie review musi mi zrobić 4 programistów w tym 2'ch seniorów dopiero wtedy QA bierze to na testy ;)

[Priya]

@mk321: Co do raportowania to różne firmy różne zwyczaje. W jednej spotkałem się z osobnymi taskami na code review, merge itp. W innej były taski tylko na projekty, które obejmowały 4h pracy dziennie, a pozostałe 4h były przeznaczone między innymi na takie rzeczy.

[wrzesien]

@mk321: Programiści między sobą, ale wymaga to pewnej organizacji. W przypadku skomplikowanych zmian można wyznaczyć kogoś kto się zna na tym temacie, zawsze też warto aby przynajmniej jeden junior spojrzał na problem.

[alex-fortune]

@mk321: pozycja architekta to dla mnie ponury żart sugerujący, że developerzy piszą kod ale nie myślą o architekturze - _ -