You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Whether this is just some strange interaction between colorama and the script command used in the test, or a bug in colorama, I'm not sure.
N.B. I had a look at how colorama works. It bills itself as a package meant only for making colour work on Windows, but I guess we are using it for its init(strip=not use_color) function that can automatically strip ANSI color chars from print() calls (it actually patches sys.stdout and sys.stderr). However there are >100 open issues and it is in a call-for-maintainers state: tartley/colorama#300.
There is a problem with this test
cylc-flow/tests/functional/cli/05-colour.t
Lines 34 to 35 in ca454d0
A color reset char is included on the last line of the output:
This is despite
autoreset=False
anduse_color=False
(strip=True
) here:cylc-flow/cylc/flow/terminal.py
Line 292 in ca454d0
Whether this is just some strange interaction between
colorama
and thescript
command used in the test, or a bug incolorama
, I'm not sure.N.B. I had a look at how colorama works. It bills itself as a package meant only for making colour work on Windows, but I guess we are using it for its
init(strip=not use_color)
function that can automatically strip ANSI color chars fromprint()
calls (it actually patchessys.stdout
andsys.stderr
). However there are >100 open issues and it is in a call-for-maintainers state: tartley/colorama#300.Originally posted by @MetRonnie in #6463 (comment)
The text was updated successfully, but these errors were encountered: