Re: [PATCH 0/15] making user-format colors conditional on config/tty

2017-07-13 Thread Junio C Hamano
Jeff King writes: > This version also takes into account feedback from the original thread. > And as I added tests, it surfaced a few corner cases around color config > that I've dealt with here. The last two patches are the most > interesting bits. > > [01/15]: check return

Re: [PATCH 0/15] making user-format colors conditional on config/tty

2017-07-13 Thread Stefan Beller
On Thu, Jul 13, 2017 at 7:55 AM, Jeff King wrote: > This is a cleanup of the patch I posted last October: > > > https://public-inbox.org/git/20161010151517.6wszhuyp57yfn...@sigill.intra.peff.net/ > > The general idea is that it's rather confusing that "%C(red)" in a >

[PATCH 0/15] making user-format colors conditional on config/tty

2017-07-13 Thread Jeff King
This is a cleanup of the patch I posted last October: https://public-inbox.org/git/20161010151517.6wszhuyp57yfn...@sigill.intra.peff.net/ The general idea is that it's rather confusing that "%C(red)" in a pretty-print format does not currently respect color.ui, --no-color, or the usual isatty