Administrace komentářů

Když dneska commitnu nefunkční kód, často vím, že je nefunkční. A nebo to přinejhorším zjistím zítra sám – není potřeba, aby to zjistili všichni, všichni to řešili nebo všichni čekali na opravu.

Pokud musím commitovat jen ucelené funkční změny, vede to k tomu, že si u sebe na disku syslím rozpracovaný necommitovaný kód. Z hlediska mergování na konci je to tedy stejné, ale ztrácím možnost průběžných commitů po jednotlivých částech, ztrácím možnost spolupráce s ostatními.

Tohle nějak nechápu - commitovat nefunkční kód je samozřejmě blbost. Idea je taková že se commituje funkční kód. A moje práce je směřována k tomu, abych za ten půl až den ten funkční kód měl. Ten čas na to stačí. Pokud budu mít rozpracovanou práci déle, vystavuju se výše popsaným problémům. Ta spolupráce je právě o tom, že se commituje. Syslení není spolupráce.

Za prvé by se nemělo běžně stávat, že si budou vývojáři navzájem přepisovat nějaké společné funkce. To by měla řešit modularita a zodpovědnost za jednotlivé moduly. Ty konfliktní změny si pak každý vyřeší sám, kdy mu to nejlépe vyhovuje.

Rozdělení zodpovědnosti za části kódu vede obecně k malé pružnosti kódu. Je lepší to mít tak, že pokud nějakej kus kódu nevyhovuje mým požadavkům tak si to tam prostě dopíšu. Když někde vidím chybu, tak ji prostě opravím. Když se mi to někde zdá nečitelné, tak refactoruju.

Takovouhle změnu je pak samozřejmě potřeba co nejdříve vypropagovat směrem k ostatním. Výhoda je, že pak všichni pracují stálým tempem, místo stylu "čekám na frantu, až do svého kódu napíše X, zatím to musím obejít a stojí mě to 5x víc času než kdybych to tam dopsal sám".

Takže např. dodělám práci, a teprve pak řeším případné konflikty. Varianta, kdy mám rozpracovanou svoji práci a do toho musím ještě řešit konflikty (a pak mám jeden commit, který řeší oboje), není dobrá. Navíc se ten případný konflikt řeší jednou při změně funkcionality – nemusí se řešit za ten měsíc třeba čtyřikrát podle toho, jak se ta změna postupně vyvíjí.

Podle mě je lepší, když do svého půldenního snažení zakomponuju kompatibilitu s půldenním snažením ostatních, místo toho, abych pracoval s měsíc starou verzí trunku a pak nezbylo než svoji mesíční práci přepsat, protože merge už nikdo nedá. (To je extrém, ale za prvé jsem to už několikrát zažil a za druhé to dobře ilustruje ten rozdíl ve stylu práce.)

Pokud někdo za 2 dny čtyřikrát něco kompletně překope a commitne, tak to asi taky nemá v hlavě úplně dobře.