Re: [PATCH] Fix various spelling errors

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: "Kekalainen, Otto" <ottoke(at)amazon(dot)com>
Cc: Michael Paquier <michael(at)paquier(dot)xyz>, Justin Pryzby <pryzby(at)telsasoft(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCH] Fix various spelling errors
Date: 2022-03-15 16:46:34
Message-ID: 328924.1647362794@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

"Kekalainen, Otto" <ottoke(at)amazon(dot)com> writes:
> Please next time use `git am` to import the patch so that author and other
> commit metadata is kept, or if you apply patches manually then commit with `git
> --author` so that original author will be correct in the commit and your name
> will be only in the committer field.

This is not our practice. We credit authors in the body of the commit
message, but we don't worry about the git metadata. git is a tool we
happen to be using at the moment, but it doesn't run the project,
and in any case it's far short of being adequate for such a purpose.
What would you do with multiple-author patches, for a start?

regards, tom lane

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2022-03-15 16:47:22 Re: [Proposal] Fully WAL logged CREATE DATABASE - No Checkpoints
Previous Message Robert Haas 2022-03-15 16:44:49 Re: Corruption during WAL replay