Re: [PATCH 2/2] bundle: use a strbuf to scan the log for boundary commits
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Feb 22, 2012, 20:50 UTC
- Message-ID
- <7vhayia8yc.fsf@alter.siamese.dyndns.org>
- In-Reply-To
- <87hayivcmm.fsf@thomas.inf.ethz.ch>
Thomas Rast <trast@inf.ethz.ch> writes:
> In any case the --pretty=oneline is very deliberate, as we can see from > the commit below. It just doesn't give a reason :-)
Thanks for digging and I hope everybody on this list who would ever send a patch learns the importance of describing _why_ in the log from this lesson.
I'll amend your 2/2 with a note that keeping the subject material is not justified and we may want to remove (or at least reduce) it in the future.