Poslední příspěvky

1

Vývoj / Re:Jak děláte code review?

« Poslední příspěvek od listoper kdy Dnes v 10:34:49 »

Ja musim znova doporucit kouknout na nejaky videa od toho Dava Farley.. a taky si treba projit reference co uvadi v description.
On to nema vycucany z prstu.
Dost casto tam zminuje a odkazuje ruzny publikace a papery a tam jsou statisticky podlozeny "dukazy" ze to funguje.

Samozrejme ze zalezi na kulture firmy, seniorite teamu atd atd.

To ze neco zni "z moji zkusenosti" divne, nebo i strasidelne neznamena nutne ze to tak je.

A kdyz postavim zkusenost jednotlivce proti vedecke praci kolektivu postavene na vzorku stovek nebo i tisicu lidi, tak "z moji zkusenosti" se vetsinou vyplati verit ty studii i kdyz ten jednotlivec bude moje moudra babicka.

A pokud treba Dave Farley neni na prvni poslech stravitelnej tak muzu jeste doporucit Kevlin Henney nebo Dan North.
Jsou o dost zabavnejsi a taky se tech temat obcas dotknou, ale mene invazivni formou...

2

Vývoj / Re:Jak děláte code review?

« Poslední příspěvek od snugar_i kdy Dnes v 10:16:14 »

Proste to zamerguju schovany za feature flag.

To ale trochu předpokládá, že všechny úpravy, co jsem dělal, jdou schovat za feature flag. Z mojí zkušenosti přidání skoro každé feature předchází refactoring, aby moc nevyčnívala. A ten taky může rozbít věci, a to věci, který už jsou "otestovaný". Takže bych čekal, že tenhle workflow povede víc ke kultuře "na fungující věci se nesahá" a kód bude rochleji nabírat dluh.

Samozřejmě záleží hodně na složení týmu a někde, kde jsou samí seniornější lidi, bych si to asi dokázal představit, pokud by se fakt hodně práce věnovalo automatickým testům (ale zase - ty taky je potřeba refaktorovat, a co když rozbiju test, a ten pak nechytí problém?)

3

Vývoj / Re:Jak děláte code review?

« Poslední příspěvek od listoper kdy Dnes v 10:06:10 »

Takze vlastne devel/main branch je stale rozbita?

Naopak... je porad releasable.

4

Vývoj / Re:Jak děláte code review?

« Poslední příspěvek od xyz kdy Dnes v 09:32:50 »

Dovolim si delegovat odpoved na na Davea... beztak to rekne lip nez ja i kdyz s nim uplne ve vsem nesouhlasim:
https://www.youtube.com/watch?v=WmVe1QrWxYU

Tak to jsem teda hodne nepochopil. To jako ze vyvinu nejakou featuru, zamerguju a pak dodatecne budu delat code review?

- minimalne se prodrazi manualni testovani. Nebo uz neni potreba pak retestovat, pokud do te featury znova sahnu?
- dodatecnym fixem muzu zanest bug (100% code coverage jeste neznamena 100% spravnost, zvlast v dobe AI)

Ja to zkusim zjednodusit, i kdyz predpokladam ze to bude moc a nekam to utece, ale treba to vysvetli tenhle dotaz...

Zkus si to predstavit tak, ze jen zmenim poradi kroku (viz priloha)

Proste to zamerguju schovany za feature flag.

Objem prace zustane stejnej jen se zmeni poradi...

No asi bych nemel klidne spani pokud bych vyvijel treba processing platebnich transakci a po zamergovani pak delal jeste nejake fixupy kvuli code review. Uprimne, podle me by to ani nikdo pricetny neschvalil takovy development proces.

Pro nejake min kriticke aplikace asi ok. Koneckoncu driv se ani code review nedelalo a ani automatizovane testy.

5

Vývoj / Re:Jak děláte code review?

« Poslední příspěvek od Pavel Tišnovský kdy Dnes v 09:15:29 »

Dovolim si delegovat odpoved na na Davea... beztak to rekne lip nez ja i kdyz s nim uplne ve vsem nesouhlasim:
https://www.youtube.com/watch?v=WmVe1QrWxYU

Tak to jsem teda hodne nepochopil. To jako ze vyvinu nejakou featuru, zamerguju a pak dodatecne budu delat code review?

- minimalne se prodrazi manualni testovani. Nebo uz neni potreba pak retestovat, pokud do te featury znova sahnu?
- dodatecnym fixem muzu zanest bug (100% code coverage jeste neznamena 100% spravnost, zvlast v dobe AI)

Ja to zkusim zjednodusit, i kdyz predpokladam ze to bude moc a nekam to utece, ale treba to vysvetli tenhle dotaz...

Zkus si to predstavit tak, ze jen zmenim poradi kroku (viz priloha)

Proste to zamerguju schovany za feature flag.

Objem prace zustane stejnej jen se zmeni poradi...

Takze vlastne devel/main branch je stale rozbita?

6

Vývoj / Re:Jak děláte code review?

« Poslední příspěvek od listoper kdy Dnes v 09:10:07 »

... tak to uživatelé nahlásí a ty to opravíš...

Vtip je predevsim v tom, ze ty nic neopravis, maximalne userum ukazes vztyceny prostrednik. A presne takhle to dela prave (a zdaleka nejen) MS.

Zrovna nedavno reportoval kolegacek bug dodavateli, na kterym bylo zjevny, ze to nikdo ani jednou ani nezkusil spustit. Meli tam syntax error v sql na kterym to ihned po spusteni zbuchlo. Pricemz se bavime o tom, ze si tohle umim odchytit a tudiz to dodavateli radne omlatit o hlavu. To beznej user neumi, tomu to proste jen pada.

Zrovna syntax error by se mel dat chytit i jinak nez ze to nekdo bude poustet...
Takze bych si vsadil, ze za tim nebude continuous delivery.

A skoro bych si vsadil na to ze to i v pohode proslo prez code review 

7

Vývoj / Re:Jak děláte code review?

« Poslední příspěvek od listoper kdy Dnes v 09:06:48 »

- minimalne se prodrazi manualni testovani. Nebo uz neni potreba pak retestovat, pokud do te featury znova sahnu?
- dodatecnym fixem muzu zanest bug (100% code coverage jeste neznamena 100% spravnost, zvlast v dobe AI)

Princip je v tom, že ti testery dělají uživatelé. Prostě spoléháš jen na automatické testy a pokud něco přesto rozbiješ, tak to uživatelé nahlásí a ty to opravíš.

Skoro... ten manual testing tam porad muzes mit (viz ten muj predchozi prispevek).
Ale urcite se daleko vic spoleha na automation.

8

Vývoj / Re:Jak děláte code review?

« Poslední příspěvek od listoper kdy Dnes v 09:02:43 »

Dovolim si delegovat odpoved na na Davea... beztak to rekne lip nez ja i kdyz s nim uplne ve vsem nesouhlasim:
https://www.youtube.com/watch?v=WmVe1QrWxYU

Tak to jsem teda hodne nepochopil. To jako ze vyvinu nejakou featuru, zamerguju a pak dodatecne budu delat code review?

- minimalne se prodrazi manualni testovani. Nebo uz neni potreba pak retestovat, pokud do te featury znova sahnu?
- dodatecnym fixem muzu zanest bug (100% code coverage jeste neznamena 100% spravnost, zvlast v dobe AI)

Ja to zkusim zjednodusit, i kdyz predpokladam ze to bude moc a nekam to utece, ale treba to vysvetli tenhle dotaz...

Zkus si to predstavit tak, ze jen zmenim poradi kroku (viz priloha)

Proste to zamerguju schovany za feature flag.

Objem prace zustane stejnej jen se zmeni poradi...

9

Vývoj / Re:Jak děláte code review?

« Poslední příspěvek od jjrsk kdy Dnes v 08:34:56 »

... tak to uživatelé nahlásí a ty to opravíš...

Vtip je predevsim v tom, ze ty nic neopravis, maximalne userum ukazes vztyceny prostrednik. A presne takhle to dela prave (a zdaleka nejen) MS.

Zrovna nedavno reportoval kolegacek bug dodavateli, na kterym bylo zjevny, ze to nikdo ani jednou ani nezkusil spustit. Meli tam syntax error v sql na kterym to ihned po spusteni zbuchlo. Pricemz se bavime o tom, ze si tohle umim odchytit a tudiz to dodavateli radne omlatit o hlavu. To beznej user neumi, tomu to proste jen pada.

10

Vývoj / Re:Jak děláte code review?

« Poslední příspěvek od L.. kdy Dnes v 07:27:38 »

- minimalne se prodrazi manualni testovani. Nebo uz neni potreba pak retestovat, pokud do te featury znova sahnu?
- dodatecnym fixem muzu zanest bug (100% code coverage jeste neznamena 100% spravnost, zvlast v dobe AI)

listoper to tady už vysvětloval. Princip je v tom, že ti testery dělají uživatelé. Prostě spoléháš jen na automatické testy a pokud něco přesto rozbiješ, tak to uživatelé nahlásí a ty to opravíš. Když si vzpomenu, jakým způsobem v poslední době funguje Microsoft, Facebook a další, tak mi přijde, že tohle workflow se opravdu používá.