search

Test-More / Test2-Harness

Alternative to Test::Harness
Other
23 stars 26 forks source link

Color everything under --color #246

Closed JRaspass closed 2 years ago

JRaspass commented 2 years ago

Not sure I fully grasp the intended logic here but this appears to be the minimum fix for #245 without regressing use without the flag or use with the --no-color flag. The logic here could almost certainly be simplified, the nested testing of $use_color at least but I feel it's all more complex than it ought to be. Suggestions welcome.

JRaspass commented 2 years ago

Also is it worth using //= for a lot of these unless defined ..., I see Test2::Harness depends on 5.10, I don't know if that was always the case.