@Trident Vasco
@Ink
Uvedu příklad: mám feature (nějaká třída), která odněkud tahá data, která sice existují z nějakého důvodu (A), ale také prostě t.č. v aplikaci nepřímo vypovídají o nějaké závislosti nebo procesech v aplikaci (B), tak k tomu kusu kódu poznačím, že předpokad aby tato dala byla k něčemu je že data (B) prostě nesou nějakou nepřímou informaci a pokud je někdo změní, tak to ovlivní fungování feature na které pracuji.
Takže když feature přestane záhadně fungovat, nebo někdo bude hledat v kódu odkaz na data (třeba skrz setter), tak ví proč a jak ta data byla použita. To se vleze na dva tři řádky (120 chars per line) přímo nad nějaký blok. A taky pak i hned může vidět která data nasměrovat místo toho, nebo že už feature data nepotřebuje nebo další upgrade.
Když pak někdo prochází Merge Requestem, tak přesně ví o co jde a pokud něco ví o datech (A) a já ne, tak už ve vchvíli MR může vidět problém nebo MR zarazit ...
To byl smysl toho mého komentáře o MR. Možná jsem to nenapsal nejlíp ...
22. 7. 2022, 15:25 editováno autorem komentáře