-
Notifications
You must be signed in to change notification settings - Fork 71
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consideration to put num_ansi_colors in an independent upstream package #747
Comments
This would indeed make sense in an ideal situation, but when it comes to (R) packaging in practice, the situation is not exactly ideal. I guess the question is mostly academic for the tinytest package, I cannot imagine that they would add a new dependency, however small. In any case, the crayon package is pretty close to what that new package would be. crayon copies the same logic as cli, and it is pretty small and lightweight, has no C code. It is superseded, so it'll never really change (except for |
I suspect you may be right, but it does seem easy enough to put this under Suggests, which is more likely to go forward. {crayon} is a great suggestion and glad to know the
I wonder if you would mind expounding on that for my own edification, not sure I see exactly what you have in mind. Optional & thank you in advance for your time & input. Closing since the request here is already satisfied. |
num_ansi_colors()
really is a wonderful piece of logic. I think other packages working with command-line interfaces would benefit from it without needing to depend on {cli}.This came up as a suggestion here:
markvanderloo/tinytest#90
WDYT, is that worthwhile to pursue?
The text was updated successfully, but these errors were encountered: