[ros-dev] Non-informational commit messages

Casper Hornstrup ch at csh-consult.dk
Sat Apr 9 09:50:23 CEST 2005



> -----Original Message-----
> From: ros-dev-bounces at reactos.com [mailto:ros-dev-bounces at reactos.com] On Behalf Of Mark Junker
> Sent: 9. april 2005 03:33
> To: ReactOS Development List
> Subject: [ros-dev] Non-informational commit messages
> 
> Hi,
> 
> it'd be nice if something like
> 
> r14554: Delete riched20
> r14555: Copy riched20
> 
> can be avoided in preference of the *cause* the required the changes above.
> 
> Everybody can see that riched20 was deleted and added afterwards but you
> cannot see *why* this has been done.
> 
> Regards,
> Mark

Commit messages should explain *what* was changed, not *why*. If there is a need
to explain why, then it should be explained in the code instead.

Casper




More information about the Ros-dev mailing list