Jeremiah Mahler <jmmah...@gmail.com> writes:
> On Fri, May 23, 2014 at 04:22:22PM -0500, Caleb Thompson wrote:
>> This patch allows people to set `commit.verbose` to implicitly send
>> +cat >check-for-no-diff <<EOF
>> +exec grep -v '^diff --git' "\$1"
>> +chmod +x check-for-no-diff
> For new tests, commands like this should be placed inside a
> test_expect_success structure. However, I can see why you did it this
> way since the code just above it does it this way.
> Perhaps others will have some recommendations.
> Also, <<\-EOF is used instead of <<EOF to remove the tabs.
> test_expect_success 'commit verbose setup' '
> cat >check-for-no-diff <<\-EOF &&
> exec grep -v '^diff --git' "\$1"
> chmod +x check-for-no-diff
Also tests use write_script these days to do this kind of thing.
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html