Jak děláte code review?

[Ink]

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

A mohl bys prosím napsat, co vidíš jinak než on?

[Reklama]

[L..]

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

V první minutě řekl takovou spoustu nesmyslů, že jsem to nevydržel a vypnul, sorry.

[listoper]

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

A mohl bys prosím napsat, co vidíš jinak než on?

Ohledne CD si ani netroufam s nim nesouhlasit... spis jsem jeste nedorostl do takove urovne abych to uplne vsechno docenil, ale spis se rozchazime v pohledu na AI.. prijde mi prilis konzervativni.

[listoper]

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

V první minutě řekl takovou spoustu nesmyslů, že jsem to nevydržel a vypnul, sorry.

njn... s tim pak ja nic nenadelam.

[xyz]

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)

[Reklama]

[L..]

- 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á.

[jjrsk]

... 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.

[listoper]

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...

[listoper]

- 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.

[listoper]

... 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 

[Pavel Tišnovský]

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?