From: Patrick Steinhardt <ps@pks.im>
Date: Mon, 05 Aug 2024 09:36:12 GMT
Subject: Re: [PATCH v3 1/6] [Newcomer] t7004-tag: redirection operator
Message-ID: <ZrCdDHqKfwWbr_Zn@tanuki>
In-Reply-To: <20240804071137.30326-2-abdobngad@gmail.com>

On Sun, Aug 04, 2024 at 10:11:32AM +0300, AbdAlRahman Gad wrote:
> Modernaize 't7004-tag.sh' by removing whitespace

s/Modernaize/Modernize. 

> after the redirect operators.

Also, please note that we typically wrap commit messages after 72
characters.

It would also be helpful to make the commit subject a bit more
informative. Right now you only say "redirection operation", which
doesn't really give the reader much of a clue what the commit does to
those. Better would be something like

    t7004: remove space after redirect operator

This gives the reader a much better signal of what the patch does.

Patrick