This is a reorganisation of the previous series, bundling the test for the fix
along with the commit itself, as suggested by Eric. It also includes many fixes
and improvements pointed out by the same reviewer, whom I thank.
The typo fix is the same as before.
In "add rename threshold tests", I in
"Felipe Gonçalves Assis" writes:
> This is a reorganisation of the previous series, bundling the test for the fix
> along with the commit itself, as suggested by Eric. It also includes many
> fixes
> and improvements pointed out by the same reviewer, whom I thank.
>
> The typo fix is the same a
On 22 February 2016 at 18:00, Junio C Hamano wrote:
> "Felipe Gonçalves Assis" writes:
>
>> This is a reorganisation of the previous series, bundling the test for the
>> fix
>> along with the commit itself, as suggested by Eric. It also includes many
>> fixes
>> and improvements pointed out by
Felipe Gonçalves Assis writes:
>> As I said, I am reluctant to take the 25%/50%/75% tests in their
>> current form. Let me take the first one and a half of the last one
>> (i.e. excluding the test) for now.
>>
>> Thanks.
>
> Ok, should I post a new version of the patch without the tests while I
On 22 February 2016 at 19:29, Junio C Hamano wrote:
> Felipe Gonçalves Assis writes:
>
>>> As I said, I am reluctant to take the 25%/50%/75% tests in their
>>> current form. Let me take the first one and a half of the last one
>>> (i.e. excluding the test) for now.
>>>
>>> Thanks.
>>
>> Ok, shou
5 matches
Mail list logo