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:
Note the section on “what is considered a patch”.
There is definitely room for improved discoverability here though.
David J.
