Stefan Beller writes:
>>> This looks good to me.
>>>
>> Thanks. Does "This" refer to 1/2 alone or the whole series?
>
> Yes. :)
>
> "This" is applicable to both patches. We had the discussion on 2/2 about me
> misreading a line a few days earlier, but apart from that it looked good, too.
Thanks
On Wed, Dec 16, 2015 at 2:41 PM, Junio C Hamano wrote:
> Stefan Beller writes:
>
>> On Thu, Dec 3, 2015 at 5:10 AM, Mike Crowe wrote:
>>> t5531 only checked that the push.recurseSubmodules config option was
>>> overridden by passing --recurse-submodules=check on the command line.
>>> Add new tes
Stefan Beller writes:
> On Thu, Dec 3, 2015 at 5:10 AM, Mike Crowe wrote:
>> t5531 only checked that the push.recurseSubmodules config option was
>> overridden by passing --recurse-submodules=check on the command line.
>> Add new tests for overriding with --recurse-submodules=no,
>> --no-recurse
On Thu, Dec 3, 2015 at 5:10 AM, Mike Crowe wrote:
> t5531 only checked that the push.recurseSubmodules config option was
> overridden by passing --recurse-submodules=check on the command line.
> Add new tests for overriding with --recurse-submodules=no,
> --no-recurse-submodules and --recurse-subm
t5531 only checked that the push.recurseSubmodules config option was
overridden by passing --recurse-submodules=check on the command line.
Add new tests for overriding with --recurse-submodules=no,
--no-recurse-submodules and --recurse-submodules=push too.
Also correct minor typo in test commit me
5 matches
Mail list logo