Re: [PATCH v3 0/3] Allow --ancestry-path to take an argument
- From
Elijah Newren <newren@gmail.com>
- Date
- Aug 20, 2022, 00:13 UTC
- Message-ID
- <CABPp-BFOag0sTO_wa8qM2h-GWigyQdj9ki0Rrd1CCKD39Y96NA@mail.gmail.com>
- In-Reply-To
- <xmqqtu68j60w.fsf@gitster.g>
On Fri, Aug 19, 2022 at 2:08 PM Junio C Hamano <gitster@pobox.com> wrote:
Show 20 quoted lines
> > Derrick Stolee <derrickstolee@github.com> writes: > > > On 8/19/2022 12:28 AM, Elijah Newren via GitGitGadget wrote: > >> Changes since v2: > >> > >> * Incorporated Stolee's suggested patch as a preliminary cleanup of t6019. > >> Slightly modified his patch by: > >> * fixing the "repetitive" typo pointed out by Eric > >> * removed the unnecessary "return 1" pointed out by Ævar > >> * switched "rev-list" to "log" since we are using --format anyway, in > >> order to remove the need to call "sed" afterward > > > > Thanks! I appreciate you fixing all of my mistakes and > > incorporating all of the existing feedback. It also looks > > nicer as your patch 2 than as a third on top. > > > > This version looks good to me. > > Yup, it looks good to me, too. Thanks, both.
And thanks to all the other reviewers as well! Good suggestions came in from several different folks.
> Let's mark it for 'next' and merge it down soonish.
Sounds good.