[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
pgpDbYDvj6EdY.pgp
Description: PGP signature
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Bug-wget] [Bug-Wget] Policy on commit messages,
Darshit Shah <=