Re: New year, new commitfest app improvements

David G. Johnston <david.g.johnston@gmail.com>

From: "David G. Johnston" <david.g.johnston@gmail.com>
To: Chao Li <li.evan.chao@gmail.com>
Cc: Jelte Fennema-Nio <me@jeltef.nl>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2026-01-14T02:51:48Z
Lists: pgsql-hackers
On Tuesday, January 13, 2026, Chao Li <li.evan.chao@gmail.com> wrote:

>
> Suppose a patch has already created a CF entry, and a reviewer wants to
> suggest a code change and attaches a diff file in the thread. In that case,
> CI will automatically pick up the diff and run tests, which will very
> likely result in a CI failure. I’ve run into this situation myself; see [1].
>
> Would it make sense for CI to only pick up .patch files and ignore .diff
> files? Or provide some way else to indicate CI to ignore certain emails?
>

Other ways exist:

https://wiki.postgresql.org/wiki/Cfbot

Note the section on “what is considered a patch”.

There is definitely room for improved discoverability here though.

David J.