On Sun, Aug 04, 2024 at 10:11:37AM +0300, AbdAlRahman Gad wrote:

changed test_expect_success that are seperated from its name like:

We do not say "changed something to something", but rather use
imperative wording as if we were instructing the code itself to change.
So maybe something like the following:

    There are several tests in t7004 where the test description that
    follows `test_expect_success` is on a separate line, violating our
    coding style. Adapt these to be on the same line.

Patrick