On Sun, Apr 7, 2013 at 12:22 PM, Junio C Hamano <gits...@pobox.com> wrote:
> Felipe Contreras <felipe.contre...@gmail.com> writes:
>> On Sun, Apr 7, 2013 at 1:42 AM, Junio C Hamano <gits...@pobox.com> wrote:
>>> Felipe Contreras <felipe.contre...@gmail.com> writes:
>>>> Some people always do --annotate, lets not force them to always type
>>>> that.
>>>> Signed-off-by: Felipe Contreras <felipe.contre...@gmail.com>
>>>> ---
>>> Sounds sensible; any tests to protect this from later breakages?
>> Given that annotate doesn't even have tests, I don't think it's needed
>> at this point.
> That is a good way to spread earlier mistakes and add more instance
> of the same kind of mistakes into the future.
> It is perfectly fine to say "I ran out my Git time for the change
> itself; help by others to add test for it is appreciated."  It is
> not reasonable to say "See there is the same mistake already in the
> code, so I am free to make the same mistake, you cannot hold it
> against me, and this feature should never be protected by tests".

I did not say that, I said I think this is not needed at this point,
if you think otherwise, you can go ahead and implement it. I think the
patch is perfectly fine as it is, and I'm not going to to add tests
for something that is potentially time consuming for me, and this
patch is not introducing, that is; the annotate feature. If we
survived without tests for annotate, we can survive without tests for

Felipe Contreras
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to