[webkit-dev] Update commit log template to add placeholder for explanation of why a patch fixes a bug
David Kilzer
ddkilzer at webkit.org
Thu Nov 17 16:42:22 PST 2022
On Nov 17, 2022, at 3:17 PM, Michael Catanzaro <mcatanzaro at gnome.org> wrote:
> On Thu, Nov 17 2022 at 02:48:04 PM -0800, Ryosuke Niwa <rniwa at apple.com> wrote:
>> But every change in WebKit comes with a Bugzilla bug.
>
> Certainly most do, but some counterexamples:
>
> * Unreviewed build fixes sometimes do not reference a bug
> * When fixing a new compiler warning or build failure, I often reference the bug that introduced the problem, rather than reporting a new one just to have a new hyperlink for the commit message
> * We probably don't need a bug report for stuff like "update my email address in contributors.json" or "fix typo in comment”
Yes, this is why I stated this in the original message:
>>> Note that the line may be simply be deleted when it doesn’t apply (such as gardening, a build fix, etc.).
I don’t think it’s too burdensome to delete a couple lines of text for the above kinds of fixes if the result is better commit messages overall.
Dave
More information about the webkit-dev
mailing list