testthat icon indicating copy to clipboard operation
testthat copied to clipboard

Published 20 hours ago • verified publisher icon r-lib

Remove crayon and use cli only

Open gaborcsardi opened this issue 3 years ago • 1 comments

In addition to less dependencies, many of the cli::ansi_*() functions work better than crayon::col_*() function, e.g. they support hyperlinks better.

gaborcsardi avatar Apr 21 '22 13:04 gaborcsardi

With this, also look at https://github.com/r-lib/testthat/blob/bb25375016ece879cecabc81fdf1d5b3072ab835/R/reporter-progress.R#L524 and replace it with the appropriate cli::ansi_nchar() call.

gaborcsardi avatar Apr 21 '22 14:04 gaborcsardi