Re: [PATCH v2 0/5] Fixes for Documentation/MyFirstObjectWalk.txt

From: Kyle Lippincott <spectral@google.com>
Date: Mar 23, 2024, 22:00 UTC
Message-ID: <p5fii3tmfb6l7dgc2p35euwv4cp6ag64qqhb63wfu75iphqknu@tdo5ru6modr3>
In-Reply-To: <cover.1710840596.git.dirk@gouders.net>

On Tue, Mar 19, 2024 at 12:23:10PM +0100, Dirk Gouders wrote:

Show 16 quoted lines

> 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.