Re: [PATCH 0/2] add format specifiers to display trailers

From: Junio C Hamano <gitster@pobox.com>
Date: Nov 18, 2016, 23:38 UTC
Message-ID: <xmqq8tsgl5o4.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <20161118230825.20952-1-jacob.e.keller@intel.com>

Jacob Keller <jacob.e.keller@intel.com> writes:

Show 5 quoted lines

> Git interpret-trailers will not recognize this as a trailer block
> because it doesn't have any standard git tags within it. Would it be ok
> to augment the trailer interpretation to say that if we have over 75%
> trailers in the block that we accept it even if it doesn't have any real
> recognized tags?

I thought the documented way to do this is to configure one of your custom trailer as such. Jonathan?

>   pretty: add %bT format for displaying trailers of a commit message

Are %(...) taken already? In longer term, it would be nice if we can unify the --pretty formats and for-each-ref formats, so it is probably better if we avoid adding any new short ones to the former.

We have %s and %b so that we can reconstruct the whole thing by using both. It is unclear how %bT fits in this picture. I wonder if we also need another placeholder that expands to the body of the message without the trailer---otherwise the whole set would become incoherent, no?