> On Nov 17, 2022, at 12:23 PM, David Kilzer via webkit-dev 
> <webkit-dev@lists.webkit.org> wrote:
> Hi,
> 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://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!).

I would remove the word “Short”. Sometimes a longer explanation is needed. More 
that one paragraph is often a good thing (para 1 explains the bug, para 2 
explains how the change fixes it).


webkit-dev mailing list

Reply via email to