Re: v3 [PATCH 2/2] status:introduce status.branch to enable --branch by default

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



<y@xxxxxxxxxxxxxxx> wrote:
> [...]

Good change.

> diff --git a/t/t7508-status.sh b/t/t7508-status.sh
> index 9a07f15..958617a 100755
> --- a/t/t7508-status.sh
> +++ b/t/t7508-status.sh

I expected one test.  Two, at most.  This is a bit overzealous.  I
don't mind leaving it as it is, but as a note for the future: this
kind of overkill is not necessarily a good thing; it makes it hard to
add/remove tests, multiple tests fail when one tiny feature doesn't
work as expected, and the testsuite takes longer to run.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]