mjs at apple.com
Wed Mar 21 14:33:15 PDT 2012
On Mar 21, 2012, at 2:29 PM, Timothy Hatcher wrote:
> Lately I have observed more and more and more changes going into WebKit that lack any details about why a particular change was made. It is intended that the ChangeLog (and commit message) contain some details about your change, not just the bug title and URL.
> The contributing information on subject is pretty sparse, which has likely caused this problem to manifest. However, the example ChangeLog linked from that page is prime example of what we all should strive for when describing our changes.
> To help curb this lack of detail in ChangeLogs, I propose we add script (or augment an existing script) to check for this missing information and inform the contributor. It is clear not all reviewers are asking patch authors to provide this information when reviewing, and such a tool would help enforce it.
I think this is a reasonable suggestion. I think we should make the contributor docs more clear about what is expected, as well.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the webkit-dev