Re: [PATCH 2/2] color: discourage use of ui.color=always

From: Jonathan Nieder <jrnieder@gmail.com>
Date: Oct 12, 2017, 04:48 UTC
Message-ID: <20171012044818.GE155740@aiede.mtv.corp.google.com>
In-Reply-To: <20171012021007.7441-3-gitster@pobox.com>

Junio C Hamano wrote:

Show 8 quoted lines

> Warn when we read such a configuration from a file, and nudge the
> users to spell them 'auto' instead.
>
> Signed-off-by: Junio C Hamano <gitster@pobox.com>
> ---
>  Documentation/config.txt | 2 +-
>  color.c                  | 7 +++++++
>  2 files changed, 8 insertions(+), 1 deletion(-)

This warning only kicks in when `always` is being silently downgraded to `auto`. It makes sense to me.

Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.