Thread (43 messages) 43 messages, 3 authors, 2025-11-03

Re: [PATCH v4 17/29] tests: t3440: create expect files at point of use

From: Li Chen <hidden>
Date: 2025-10-15 13:59:14

Hi Kristoffer,

Thanks for the review suggestions! I'll address them in the next version.

 ---- On Wed, 15 Oct 2025 04:41:33 +0800  Kristoffer Haugsbakk [off-list ref] wrote --- 
 > Now you start to change the test suite/file that you created for this
 > series.  There shouldn’t be a need to do a test file-only patch/commit
 > for a fresh series.
 > 
 > I saw in one of your patches that you removed `--keep-empty` from a test
 > because “that is the default”.  I also saw Phillip’s comment somewhere
 > that said the same thing.
 > 
 > The goal with maturing series is not to add patches on top in each round
 > (if that’s what you are doing).  It is to recreate them as if the series
 > was perfectly written to begin with; if one patch introduces
 > `--trailers` and a test file, then there should be no need with
 > follow-up patches that improve the test file style, refactors it, and
 > so on.

Thanks for the tip. I split the changes into separate commits to ease review,
 as Phillip suggested in https://lore.kernel.org/git/d4c9f082-52be-48d9-b817-fcb8a72e1bd7@gmail.com/ (local).

It seems I may have overdone it? If so, I'll try for a better balance in the next version.

Li​
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help