Issue #12251 has been updated by Pieter van de Bruggen.
> This is a behavioural change for this code, compared to other 2.7.x code. Has > a decision been made about the value of that, compared to the cost of > difference, yet? This face is acting as a design prototype for the command line conventions we will be promoting in Telly. >> Colors should be supressed if the output stream is not a TTY. > This should follow the behaviour of the current log system. This will follow the behaviour of the log system (specifically, responding to `Puppet[:color]`), with the expectation that system will become more TTY-aware in Telly. >> Messages sent to STDERR should be bright red. > This is a wild divergence, where previously parts of Puppet could print to > these output channels without any colour applied at all, and only log > messages were coloured. This is intended only to apply to logged messages, though we already generally discourage direct output to STDOUT and STDERR. >> Nice to have: if the output stream is not a TTY, default to YAML or JSON >> output. > That last should be a separate ticket, tracking a uniform change to Faces. It > is an API breaking change, so shouldn’t apply inside the stable series, only > in Telly, and should apply uniformly to everything built on faces. I can think of no reason to assume this would necessarily be an API breaking change (*unless* it were uniformly applied to all faces), but as it stands, this is presently nowhere near the roadmap. ---------------------------------------- Feature #12251: Colorized puppet module list output https://projects.puppetlabs.com/issues/12251#change-54582 Author: Matt Robinson Status: Needs Decision Priority: Normal Assignee: Pieter van de Bruggen Category: Target version: 2.7.x Affected Puppet version: Keywords: geordi Branch: -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-bugs?hl=en.
