Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH v5 3/3] Color skipped tests bold blue

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:54:49

Stefano Lattarini [off-list ref] writes:
Hi Adam.

On 09/20/2012 11:08 AM, Adam Spiers wrote:
quoted
Skipped tests indicate incomplete test coverage.  Whilst this is
not a test failure or other error, it's still not complete
success, so according to the universal traffic lights coloring
scheme, yellow/brown seems more suitable than green.  However,
it's more informational than cautionary, so instead we use blue
which is a universal color for information signs.  Bold blue
should work better on both black and white backgrounds than
normal blue.
A very minor nit (feel free to ignore it): IMHO, it should be nice
to state explicitly in the commit message that blue is already used
by other testsuite-related software to highlight skipped tests; you
can report the examples of at least Automake, Autotest and prove --
and extra kudos if you find further examples ;-)
Thanks.  Will tweak the proposed log message to include the above
when queuing the updated patch.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help