Thread (2 messages) 2 messages, 2 authors, 2018-04-05

Re: [PATCH v3 3/3] builtin/config: introduce `color` type specifier

From: Junio C Hamano <hidden>
Date: 2018-03-30 18:09:51

Taylor Blau [off-list ref] writes:
quoted hunk
@@ -184,6 +183,7 @@ Valid `[type]`'s include:
 --bool-or-int::
 --path::
 --expiry-date::
+--color::
   Historical options for selecting a type specifier. Prefer instead `--type`,
   (see: above).
 
@@ -223,6 +223,9 @@ Valid `[type]`'s include:
 	output it as the ANSI color escape sequence to the standard
 	output.  The optional `default` parameter is used instead, if
 	there is no color configured for `name`.
++
+It is preferred to use `--type=color`, or `--type=color --default=[default]`
+instead of `--get-color`.
Wasn't the whole point of the preliminary --type=<type> patch to
avoid having to add thse two hunks?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help