From: Junio C Hamano Date: Sun, 07 Apr 2013 17:22:44 GMT Subject: Re: [PATCH v3 1/4] send-email: make annotate configurable Message-ID: <7vvc7y45nf.fsf@alter.siamese.dyndns.org> In-Reply-To: Felipe Contreras writes: > On Sun, Apr 7, 2013 at 1:42 AM, Junio C Hamano wrote: >> Felipe Contreras writes: >> >>> Some people always do --annotate, lets not force them to always type >>> that. >>> >>> Signed-off-by: Felipe Contreras >>> --- >> >> 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".