Thread (14 messages) 14 messages, 3 authors, 2024-08-05

Re: [PATCH v3 3/6] [Newcomer] t7004-tag: one command per line

From: Patrick Steinhardt <hidden>
Date: 2024-08-05 09:36:52

On Sun, Aug 04, 2024 at 10:11:34AM +0300, AbdAlRahman Gad wrote:
fix lines like these:

git tag -l >actual && test_cmp expect actual &&

to

git tag -l >actual &&
test_cmp expect actual &&
Same remark here regarding the style of this commit message. I also
think that the message is both overly verbose and not telling enough at
the same point in time, because you require the reader to implicitly
derive what the change is that you are doing. Proposal:

    One of the tests in t7004 has multiple commands on a single line,
    which is discouraged. Adapt these by splitting up these into one
    line per command.

Patrick

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help