[webkit-dev] Update commit log template to add placeholder for explanation of why a patch fixes a bug
Ryosuke Niwa
rniwa at apple.com
Thu Nov 17 14:41:35 PST 2022
> On Nov 17, 2022, at 2:37 PM, Michael Catanzaro via webkit-dev <webkit-dev at lists.webkit.org> wrote:
>
> On Thu, Nov 17 2022 at 12:23:54 PM -0800, David Kilzer via webkit-dev <webkit-dev at lists.webkit.org> wrote:
>> Any feedback on this change?
>
> We could alternatively say "Explanation of this change (OOPS!)" or "Explanation of this commit (OOPS!)" to be a little more general.
That’s strictly worse than “explain why this fixes the bug”.
We don’t want a description of what PR is; that’s obvious from diff. We want a description of why that PR fixes the bug.
- R. Niwa
More information about the webkit-dev
mailing list