Re: [PATCH 1/8] Use %B for Split Subject/Body
From: Junio C Hamano <hidden>
Date: 2016-06-15 22:55:38
greened@obbligato.org writes:
Ack, of course. I don't know how I missed that.quoted
quoted
# 15 test_expect_success 'add main6' ' create main6 &&Why?It was in the original testsuite from Avery. I didn't add or remove any tests when I first integrated git-subtree.
The question was about the lossage of the blank line, which does not seem to be related to what this patch wants to do.
quoted
quoted
-# 25 +#25Why the lossage of a SP?I think this got fixed later in the series.
That is not a good excuse to introduce breakages in the first place, no?
quoted
It may make sense to lose these "# num" that will have to be touched every time somebody inserts new test pieces in the middle, as a preparatory step before any of these patches, by the way. That will reduce noise in the patches for real changes.Yeah, I know, but it makes it really easy to find a test when something goes wrong.
That is what "tXXXX-*.sh -i" is for, isn't it?