On Sun, Oct 06, 2024 at 04:26:03PM +0000, Usman Akinyemi wrote:

Thanks for this. I will definitely take note of this next time. Thank you.

Please note that we tend to not top-post on this mailing list. Instead,
your reply should be either inline in the quoted parts in case you want
to reply to individual parts of it, or below the quoted part as others
are doing :)

Patrick

On Sun, Oct 6, 2024 at 4:21 PM Kristoffer Haugsbakk
[off-list ref] wrote:

quoted

On Sun, Oct 6, 2024, at 18:06, Usman Akinyemi via GitGitGadget wrote:

quoted

The root cause of the whitespace issue was quoting $count in the test
command, which led to the inclusion of whitespace in the comparison. By
removing the quotes around $count, the comparison works as expected without
the need for tr -d.

Signed-off-by: Usman Akinyemi

Usman Akinyemi (2):
  t3404: avoid losing exit status with focus on `git show` and `git
    cat-files`
  [Outreachy][Patch v1] t3404: employing test_line_count() to replace
    test

You don’t have to sign off on your cover letter.  Just the patches. :)

Of course do what you prefer.