From: Patrick Steinhardt <ps@pks.im>
Date: Mon, 05 Aug 2024 09:42:01 GMT
Subject: Re: [PATCH v3 5/6] [Newcomer] t7004-tag: here-doc modernization
Message-ID: <ZrCeaU4VKxPYTPiJ@tanuki>
In-Reply-To: <20240804071137.30326-6-abdobngad@gmail.com>

On Sun, Aug 04, 2024 at 10:11:36AM +0300, AbdAlRahman Gad wrote:
> use <<-\EOF to indent here-doc

Instead of what? It would help to mention that the change converts from
<<\EOF to <<\-EOF, where the latter allows us to indent the body.

I'd also suggest a commit title like "t7004: use indented here docs", as
"here doc modernizations" doesn't quite give you a hint what the
modernization looks like.

Patrick