From: Junio C Hamano <gitster@pobox.com>
Date: Fri, 13 Jun 2025 16:26:18 GMT
Subject: Re: [PATCH v2 1/2] merge/pull: --compact-summary
Message-ID: <xmqq5xgza9qt.fsf@gitster.g>
In-Reply-To: <857ea4d4-2252-4d46-9f93-e78e9450bb8c@app.fastmail.com>

"Kristoffer Haugsbakk" <kristofferhaugsbakk@fastmail.com> writes:

> On Fri, Jun 13, 2025, at 00:25, Junio C Hamano wrote:
>>  static struct option builtin_merge_options[] = {
>>  	OPT_SET_INT('n', NULL, &show_diffstat,
>>  		N_("do not show a diffstat at the end of the merge"), 0),
>>  	OPT_BOOL(0, "stat", &show_diffstat,
>>  		N_("show a diffstat at the end of the merge")),
>>  	OPT_BOOL(0, "summary", &show_diffstat, N_("(synonym to --stat)")),
>> +	OPT_CALLBACK_F(0, "compact-summary", &show_diffstat, N_("compact-summary"),
>> +		       N_("show a compactstat at the end of the merge"),
>> +		       PARSE_OPT_NOARG,
>> +		       option_parse_compact_summary),
>
> `compactstat` is very dense.  s/compactstat/compact stat/ ?

Nice spotting.  I think "compact summary" (to match the diff option
"--compact-summary") may be even better.