Re: [PATCH] merge/pull: --compact-summary
- From
D. Ben Knoble <ben.knoble@gmail.com>
- Date
- Jun 11, 2025, 23:26 UTC
- Message-ID
- <CALnO6CCrCcEp1FWWWPQ2fts-n-jHZdfYWsyYsG9W5aUV42gs8Q@mail.gmail.com>
- In-Reply-To
- <xmqqzfefj7j3.fsf@gitster.g>
On Tue, Jun 10, 2025 at 11:12 PM Junio C Hamano <gitster@pobox.com> wrote:
Show 20 quoted lines
> > "D. Ben Knoble" <ben.knoble@gmail.com> writes: > > > Does this suggest that we either want a "diff options" option (or > > config), which would be more general? > > If I have bk/new-topic branch, here is how I would merge it. > > $ git merge --compact-summary bk/new-topic > > but there is an existing merge.stat that (1) is a mere boolean and > (2) is true by default to force use of "--stat" option. We could > extend it to "bool or the string 'compact-summary'" and when it is > > - not set, do the "--stat" as before; > - set to true, do the "--stat" as before; > - set to 'compact-summary', do the "--compact-summary". > > With such a configuration variable, "git merge bk/new-topic" would > be how I would run my merge.
Makes sense; I suppose my question was more along the lines of "should we go ahead and support extending the options we give to diff more broadly rather than add a one-off flag (and potentially add more later)."
-- D. Ben Knoble