Jak děláte code review?

[Jiří Havel]

Takze vlastne devel/main branch je stale rozbita?

Naopak... je porad releasable.

On je tam nezanedbatelný překryv. A záleží primárně na podmínkách ve smlouvách, míře monopolního postavení a dalších nesoftwarových věcech.
Standardní euly pro plebs se přece píšou tak, aby bylo releasnutelné úplně cokoliv.

[Reklama]

[Pavel Tišnovský]

Takze vlastne devel/main branch je stale rozbita?

Naopak... je porad releasable.

To jo, ale ptal jsem se na hlavni vetev. Tam po mergi budu ji budu mit ihned rozbitou ne?

[listoper]

Takze vlastne devel/main branch je stale rozbita?

Naopak... je porad releasable.

To jo, ale ptal jsem se na hlavni vetev. Tam po mergi budu ji budu mit ihned rozbitou ne?

Ja taky mluvim o hlavni vetvi... idealne jedinne vetvi... Proc by mela byt rozbita?

[Pavel Tišnovský]

Takze vlastne devel/main branch je stale rozbita?

Naopak... je porad releasable.

To jo, ale ptal jsem se na hlavni vetev. Tam po mergi budu ji budu mit ihned rozbitou ne?

Ja taky mluvim o hlavni vetvi... idealne jedinne vetvi... Proc by mela byt rozbita?

No podle toho obrázku se nějaký review řeší až po merge. Ale tuším, že mi uniká nějaká hlavní myšlenka tohoto workflow.

https://forum.root.cz/index.php?action=dlattach;topic=31593.0;attach=8122;image

[listoper]

No podle toho obrázku se nějaký review řeší až po merge. Ale tuším, že mi uniká nějaká hlavní myšlenka tohoto workflow.

https://forum.root.cz/index.php?action=dlattach;topic=31593.0;attach=8122;image

Ja to vedel, ze kdyz to zkusim zjednodusit tak to nedopadne.

Zkusim doplnit v poznamkach:
1. Pair/Mob programming > code review => tzn idealne probiha "code review" soubezne s developmentem
2. Pokud to projde vsema automatizovanejma testama tak to neni rozbity a muze to jit na produkci, ataky to tam jde... treba 10x denne (mozna schovany za feature flagem)
3. Pokud je to "rozbity" i kdyz to proslo pipelinou tak musim vylepsit pipelinu ;-)

Asi zase doplnim odkaz https://www.youtube.com/watch?v=CR3LP2n2dWw

[Reklama]

[Pavel Tišnovský]

No podle toho obrázku se nějaký review řeší až po merge. Ale tuším, že mi uniká nějaká hlavní myšlenka tohoto workflow.

https://forum.root.cz/index.php?action=dlattach;topic=31593.0;attach=8122;image

Ja to vedel, ze kdyz to zkusim zjednodusit tak to nedopadne.

Zkusim doplnit v poznamkach:
1. Pair/Mob programming > code review => tzn idealne probiha "code review" soubezne s developmentem
2. Pokud to projde vsema automatizovanejma testama tak to neni rozbity a muze to jit na produkci, ataky to tam jde... treba 10x denne (mozna schovany za feature flagem)
3. Pokud je to "rozbity" i kdyz to proslo pipelinou tak musim vylepsit pipelinu ;-)

Asi zase doplnim odkaz https://www.youtube.com/watch?v=CR3LP2n2dWw

ok to už začíná dávat smysl, dokážu si i představit, jak by mělo vypadat složení takového týmu. Dík