bug-wget
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Bug-wget] [Bug-Wget] Policy on commit messages


From: Darshit Shah
Subject: [Bug-wget] [Bug-Wget] Policy on commit messages
Date: Sat, 1 Nov 2014 05:43:11 +0530
User-agent: Mutt/1.5.23 (2014-03-12)

Hi,

In the last few days we've seen a huge flurry of activity and a large number of patches have been committed to the code base. As someone who was traveling and hence unable to keep up with the mailing lists, I tried to catch up by looking at the code base and the changes that have been made.

It would turn out that the commit messages aren't very informative on the last few commits. Unless I know the context of the change from the mailing list, it would be very difficult to understand why that particular change was important. Both the ChangeLog and commit messages usually only describe the change in literal terms but not in their logical reasoning.

I propose that henceforth, all commits be reviewed for detailed descriptions of not only the syntactical changes, but also the logical reasons behind the change. This will help us in the long term to follow through the changes in the code base and also make it easier for new contributors to understand why some things are the way they are.

--
Thanking You,
Darshit Shah

Attachment: pgpDbYDvj6EdY.pgp
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]