[PATCH] D152285: Add support for the NO_COLOR environment variable

2023-06-07 Thread Jan Niklas Hasse via Phabricator via cfe-commits
jhasse added a comment. In D152285#4403031 , @aaron.ballman wrote: > In D152285#4401348 , @MaskRay wrote: > >> If we don't intend to support both standards, we can close >> https://github.com/llvm/llvm-project/i

[PATCH] D152285: Add support for the NO_COLOR environment variable

2023-06-07 Thread Jan Niklas Hasse via Phabricator via cfe-commits
jhasse added a comment. There's a valid usecase `CLICOLOR_FORCE`: Force color diagnostics. The "disable colors" part of https://bixense.com/clicolors/ is not that important to me, I could change it to point to `NO_COLOR` instead? btw: I've tried to join the "standards" a few years ago: https:/

[PATCH] D41720: [clang-tidy] Add a -show-color flag.

2019-02-21 Thread Jan Niklas Hasse via Phabricator via cfe-commits
jhasse added a comment. What are you thoughts about using an environment variable for this? Due to the number of differently named flags, it's often hard to configure a CI system to display color in every tool. (I'm trying to push for a standard for this, see https://bixense.com/clicolors/) C