From: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date: Sun, 25 Jul 2010 17:46:27 GMT
Subject: Re: [PATCH v2 0/7] Detailed test coverage reports for Git
Message-ID: <AANLkTikHXRJcuv35uFrv_g1umVdUM-QD80brFGj94E3N@mail.gmail.com>
In-Reply-To: <20100725172012.GC8604@burratino>

On Sun, Jul 25, 2010 at 17:20, Jonathan Nieder <jrnieder@gmail.com> wrote:
> Ævar Arnfjörð Bjarmason wrote:
>
>> Ævar Arnfjörð Bjarmason (7):
>>   gitignore: Ignore files generated by "make coverage"
>>   Makefile: Include subdirectories in "make cover" reports
>>   Makefile: Split out the untested functions target
>>   Makefile: Add coverage-report-cover-db target
>>   Makefile: Add coverage-report-cover-db-html target
>>   t/README: A new section about test coverage
>>   t/README: Add a note about the dangers of coverage chasing
>
> With whatever subset of the changes I have hinted at seems
> suitable,
>
> Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>

All the changes you made look good, I approve of having them squashed
when this is applied. Thanks.