On 2022-12-14 16:21, Junio C Hamano wrote: > Victoria Dye <vdye@xxxxxxxxxx> writes: > >> A general comment about this series - the way you have the patches organized >> means that the "feature" content you're trying to integrate (the first two >> patches) is contextually separated from these tests. For people that >> learn/understand code via examples in tests, this makes it really difficult >> to understand what's going on. To avoid that, I think you could rearrange >> the patches pretty easily: >> ... > > Thanks for a thorough review of the entire series, with concrete > suggestions for improvements with encouragements sprinkled in. > > Very much appreciated. > Yes! Thank you Victoria for the detailed and thorough review. I also too very much appreciate it :-) Thanks, Matthew
