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

[to_je_jedno]

Objem prace zustane stejnej jen se zmeni poradi...

Tohle je jen teorie. Code review po mergi je asi hezky predpoklad, ale nebude ve vetsi mire fungovat.

[listoper]

Objem prace zustane stejnej jen se zmeni poradi...

Tohle je jen teorie.

Vsak jsem psal ze je to zjednodusene a jen na vysvetleni toho jednoho pointu...

Code review po mergi je asi hezky predpoklad, ale nebude ve vetsi mire fungovat.

Dave Farley rika, ze jo a doklada to nekolika case studies kde uz jim to zafungovalo a tim padem ja uz nemusim nic dokazovet 

[Zdenek Tomes]

Dave Farley rika, ze jo a doklada to nekolika case studies kde uz jim to zafungovalo a tim padem ja uz nemusim nic dokazovet 

Tak jeho prace je mit videa generujici penize, takze bych ho na 100% nebral vazne. Proste udela viralni video a o par mesicu pozdeji to otoci (kdysi takto otacel ohledne mikrosluzeb).

[listoper]

... a doklada to nekolika case studies ...

[to_je_jedno]

Dave Farley rika, ze jo a doklada to nekolika case studies kde uz jim to zafungovalo a tim padem ja uz nemusim nic dokazovet 

Nekolik case studies != ve vetsi mire.

Stejne jako existence nekolika tyrkysovych nerika nic o tom, ze by to melo fungovat ve vetsi mire.