> On Nov 17, 2022, at 12:23 PM, David Kilzer via webkit-dev 
> <webkit-dev@lists.webkit.org> wrote:
> 
> The following PR adds placeholder text in the commit log template to remind 
> authors to explain why a change fixes a bug:
> 
> Bug 248012: Update commit message template to request a brief explanation of 
> why a PR fixes the bug
> <https://bugs.webkit.org/show_bug.cgi?id=248012 
> <https://bugs.webkit.org/show_bug.cgi?id=248012>>
> <https://github.com/WebKit/WebKit/pull/6579 
> <https://github.com/WebKit/WebKit/pull/6579>>
> 
> It looks like this:
> 
> Need a short description (OOPS!).
> Need the bug URL (OOPS!).
> Include a Radar link (OOPS!).
> 
> Reviewed by NOBODY (OOPS!).
> 
> Short explanation why this fixes the bug (OOPS!).
> 
> * path/to/File.cpp:
> (Method::Name):
> 
> Since this is effectively a policy change, Jonathan suggested I post it here 
> first before the change is landed.
> 
> I believe many WebKit contributors already do this anyway, and it’s always 
> been a best-practice to explain why a change fixes a bug (vs. what a patch 
> does, which is usually obvious by reading the code), so this text just serves 
> as a reminder to write a short explanation for every relevant patch.
> 
> Note that the line may be simply be deleted when it doesn’t apply (such as 
> gardening, a build fix, etc.).
> 
> Any feedback on this change?

Seems like a good idea.

- R. Niwa

_______________________________________________
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev

Reply via email to