Dirk Gouders <dirk@xxxxxxxxxxx> writes: > Kyle Lippincott <spectral@xxxxxxxxxx> writes: > >> On Tue, Mar 19, 2024 at 12:23:10PM +0100, Dirk Gouders wrote: >>> The second spin for this series. >>> --- >>> Changes since v1: >>> * Added Emily to Cc in the hope for a review >>> * Remove superfluous tags from [1/5] and [3/5] >>> * Replace bashism `|&` by `2>&1 |` in [5/5] >>> --- >>> Dirk Gouders (5): >>> MyFirstObjectWalk: use additional arg in config_fn_t >>> MyFirstObjectWalk: fix misspelled "builtins/" >>> MyFirstObjectWalk: fix filtered object walk >>> MyFirstObjectWalk: fix description for counting omitted objects >>> MyFirstObjectWalk: add stderr to pipe processing >>> >>> Documentation/MyFirstObjectWalk.txt | 36 ++++++++++++++++------------- >>> 1 file changed, 20 insertions(+), 16 deletions(-) >> >> Aside from the small comments on 4 and 5, series looks good to me, thanks for >> working on this. > > Thanks for the review -- especially for the detailed explanation and > suggestions on 4. Yeah, I too liked the comments on [4/5]. Thanks for working well together.
