From: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date: Sun, 25 Jul 2010 19:27:19 GMT
Subject: Re: [PATCH v2 7/7] t/README: Add a note about the dangers of coverage chasing
Message-ID: <AANLkTilcivnC1ZIZ1vDKmMZwCc7Hdc9gSTNYzQEYM08z@mail.gmail.com>
In-Reply-To: <20100725160545.GA25169@burratino>

On Sun, Jul 25, 2010 at 16:05, Jonathan Nieder <jrnieder@gmail.com> wrote:
> Ævar Arnfjörð Bjarmason wrote:
>
>> +   Don't blindly follow test coverage metrics
>
> Hmph, that is just common sense,

You'd be surprised at how uncommon it is when people have 98% coverage
and try to painfully squeeze out that last 2% :)

> while “you should really not be paying any attention to your code
> while writing tests” is not.  I even prefer the text without this
> patch applied.  So forget I said anything; I can find a way to hint
> at that in t/README later. :)

I don't know whether it should be applied. I just wrote a short
summary in response to the previous commentary.