action #114878
closedisotovideo: Add option to disable color output size:S
Description
Updated by tinita over 2 years ago
- Copied from action #114869: [qe-core] os-autoinst-distri-opensuse pipeline is broken added
Updated by mkittler over 2 years ago
Should be easy to do although I'm wondering why setting the env variable isn't sufficient.
Updated by okurz over 2 years ago
Because nobody will find the environment variable but if command --help
states it then it will be used :)
Updated by okurz over 2 years ago
- Status changed from New to In Progress
- Assignee set to okurz
I will try to work on this during today's workshop day
Updated by openqa_review over 2 years ago
- Due date set to 2022-08-18
Setting due date based on mean cycle time of SUSE QE Tools
Updated by livdywan over 2 years ago
- Subject changed from isotovideo: Add option to disable color output to isotovideo: Add option to disable color output size:S
- Description updated (diff)
Updated by livdywan over 2 years ago
- Assignee changed from okurz to livdywan
isotovideo implicitly supports ANSI_COLORS_DISABLED and NO_COLOR. We can actually use this opportunity to document it, in addition to implementing a command-line switch.
Updated by livdywan over 2 years ago
After discussing it in the unblock we decided to omit the "auto" option for now and @okurz already proposed a branch with just --color=yes|no support which doesn't require openQA changes. It was bugging me that we couldn't sort out the tests but I'll leave the drafts, and we can re-visit them later.
Updated by livdywan over 2 years ago
- Status changed from In Progress to Feedback
cdywan wrote:
My branch has retroactively become the follow-up branch for color=auto which needs to use IO::Interactive to make it unit-testable. The AC's are fulfilled so the ticket isn't dependent on this being merged.
Updated by okurz over 2 years ago
I agree that the AC's are covered. So I suggest you decide between the following alternatives:
- Close the draft PRs and resolve this ticket
- Work on the draft PRs
Updated by livdywan over 2 years ago
I'm resolving the ticket because it looks like unit testing is not trivial afterall and that feels like it's dragging on too much.
Enabling and disabling colors via --color=yes
and --color=no
meanwhile to work nicely.