Fix Version/s: None
There's code in pex.config.history to colour output, but it's en/disabled unconditionally. Please change it to never colour text that isn't going to the terminal.
- mentioned in
Please document the change in the doc string for the Color.colorize method. Other than that it looks fine.
As an aside: I am surprised that Color._colorize is a class variable instead of an instance variable, and similarly that Color.colorize is static. I wish the class had a doc string, but it's out of scope to ask you to add one.
Done; I also added a class doc string.
I think it makes sense for that to be a class variable – it's state about how you want to use colour. It would be nice to add 'with' support, but I think that's out of scope.
Trivial change, needed to make
DM-5421more user friendly