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.