Subject: Re: [PATCH v5 3/5][Outreachy] t7102,t7201: remove whitespace after redirect operator

From: Junio C Hamano <[email-masked]>
Date: Thu, 22 Oct 2020 10:35:04 -0700

Eric Sunshine <[email-masked]> writes:

> On Thu, Oct 22, 2020 at 1:56 AM Junio C Hamano <[email-masked]> wrote:
>> Subject: t7102: prepare expected output inside test_expect_* block
>>
>> Signed-off-by: Junio C Hamano <[email-masked]>
>> ---
>> diff --git c/t/t7102-reset.sh w/t/t7102-reset.sh
>> @@ -82,15 +82,15 @@ test_expect_success 'reset --hard message (ISO8859-1 logoutputencoding)' '
>> ->.diff_expect
>> ->.cached_expect
>> -cat >.cat_expect <<EOF
>> -secondfile:
>> -EOF
>> -
>>  test_expect_success 'giving a non existing revision should fail' '
>> +       >.diff_expect &&
>> +       >.cached_expect &&
>> +       cat >.cat_expect <<-\EOF &&
>> +       secondfile:
>> +       EOF
>
> You used <<-\EOF rather than plain <<-EOF when possible. Good.
>
> (Might be worth mention in the commit message, but perhaps too minor?)
>
>> @@ -191,38 +191,38 @@ test_expect_success 'resetting to HEAD with no changes should succeed and do not
>>  test_expect_success '--soft reset only should show changes in diff --cached' '
>> +       >.diff_expect &&
>> +       cat >.cached_expect <<-EOF &&
>> +       diff --git a/secondfile b/secondfile
>> +       index $head5p1s..$head5s 100644
>
> And used plain <<-EOF when necessary. Fine.

Yup.

Let's declare victory with v5 plus this one as [patch 6/5], and move
on.

We both know very well that through the microproject that it has
been demonstrated that Charvi can now comfortably work with us in
the review discussions.

Thanks, both.