Ho notato che recentemente ho scritto il seguente messaggio di commit:
Since HTML changed, getting the index would always result in zero, since the element is now enclosed in a parent block.
Per chi inizia visualizzando per prima cosa il diff
del commit commit (ad esempio dopo aver eseguito blame
su un file) e non ottiene lo scopo della modifica, questo messaggio è utile:
-
Indica perché è stata eseguita la modifica all'interno del commit, non quale era la modifica (un
diff
direbbe a che la modifica in realtà è). -
Spiega l'immagine più grande, il contesto in cui è stato eseguito il commit.
Tuttavia, per qualcuno che sta semplicemente sfogliando il log, il messaggio è assolutamente criptico. Quale HTML? Quale indice? Quale blocco genitore? Cosa diavolo sta succedendo qui? Per capire questo, la persona dovrebbe avere bisogno di diff
delle modifiche e solo allora capirà il messaggio. Non importa nemmeno se la persona conosce il progetto; Sono abbastanza sicuro di non essere in grado di spiegare il messaggio da solo in una settimana senza consultare i risultati di diff
.
Un messaggio di log di un commit dovrebbe essere esplicito da solo, senza che uno sviluppatore debba esplorare il diff
per comprenderlo?