Re: [PATCH v2 00/10] Modernizing the t7001 test script
- From
Shubham Verma <shubhunic@gmail.com>
- Date
- Feb 11, 2021, 07:56 UTC
- Message-ID
- <CAD=kLpvfAz7wkvBfakbc6woNWdyOxrYn0XKgSPkmTNz2RCNkKw@mail.gmail.com>
- In-Reply-To
- <xmqq35y3ys0w.fsf@gitster.c.googlers.com>
Thank You Eric and Junio.
Sorry I forget about spelling. Okay I will correct it and send a new patch series today.
Thank You!
On Thu, Feb 11, 2021 at 1:07 PM Junio C Hamano <gitster@pobox.com> wrote:
Show 23 quoted lines
> > Eric Sunshine <sunshine@sunshineco.com> writes: > > > On Tue, Feb 9, 2021 at 6:13 PM Junio C Hamano <gitster@pobox.com> wrote: > >> shubham verma <shubhunic@gmail.com> writes: > >> > In this patch series modernize the t7001 test script by changing the > >> > style of its tests from an old one to the modern one and by cleaning > >> > up the test script. > >> > > >> I would have used test_write_lines instead of here-doc in 9/10, but > >> that is such a minor preference thing that it is not worth going > >> back and replace. Nothing else stood out during my cursory scan. > > > > I read over the entire series and likewise didn't find anything to > > comment on, thus consider it in good shape. For what it's worth, > > consider this series: > > > > Reviewed-by: Eric Sunshine <sunshine@sunshineco.com> > > Yup, this one looked good to me. Thanks. > > Except that the author's name were spelled in all lowercase, unlike > v1, which made me wonder if it was intentional, that is.