Re: [PATCH v2 1/2] merge/pull: --compact-summary
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jun 13, 2025, 16:26 UTC
- Message-ID
- <xmqq5xgza9qt.fsf@gitster.g>
- In-Reply-To
- <857ea4d4-2252-4d46-9f93-e78e9450bb8c@app.fastmail.com>
"Kristoffer Haugsbakk" <kristofferhaugsbakk@fastmail.com> writes:
Show 13 quoted lines
> 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.