Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH v2] log: make --abbrev-commit's ellipsis configurable

From: Jakub Narebski <hidden>
Date: 2016-06-15 22:46:10

Johannes Sixt [off-list ref] writes:
Thomas Rast schrieb:
quoted
So we introduce a new variable format.abbrevEllipsis that defaults to
true (previous behaviour).
How about format.hideAbbrevDots that defaults to false? Then you can write
in the config file:

    [format]
         hideAbbrevDots

(note: without '= true').

Even though I usually detest double negations (and I would count
hideAbbrevDots=false as one), the ability to set the non-default value of
a Boolean variable in this way trumps my taste.
By the way, should it be in [format] section? I thought that format.*
variables are about git-format-patch... Wouldn't it be better to put it
in [log] section?

On the other hand we have already format.pretty (but log.date and
log.showroot).  Nice consistency ;-(


--
Jakub Narebski
Poland
ShadeHawk on #git
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help