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
CYAN (called blue) is used for the man.dvc.org URLs here and here. Not sure we need this per the first bullet (BLUE).
MAGENTA for 2 "HINT" messages here and here. Should check it's used in all such hints, and that other "SOMETHING:" messages always have heir own colors.
Finally, PR Improve LockError message #2765 may introduce BLUE for sample commands in the output (those surrounded with back ticks for example. Should we apply this everywhere? (Also "dynamic" commands, printed inside single quotes'`.)
I just recently discovered we're using
coloramawhen reviewing https://github.com/iterative/dvc/pull/2765/files but haven't really noticed it much in most DVC output.Current use I can see now:
colorama.Fore.BLUEis used for some URLs. I like this, should check it's ALL URLs.REDandGREENare used in the updater. This seems OK.BLUE,RED, andYELLOWin the logger.CYAN(calledblue) is used for the man.dvc.org URLs here and here. Not sure we need this per the first bullet (BLUE).MAGENTAfor 2 "HINT" messages here and here. Should check it's used in all such hints, and that other "SOMETHING:" messages always have heir own colors.colorizefunction in dvc/dvc/utils/init.py:https://github.com/iterative/dvc/blob/cfc442e932701af5ef94b94d8b70b8bd117fc6f8/dvc/utils/__init__.py#L349-L359
but doesn't seem to be in use. Use it or lose it.only used in boxify. Use it more (or remove it).LockErrormessage #2765 may introduceBLUEfor sample commands in the output (those surrounded with back ticksfor example. Should we apply this everywhere? (Also "dynamic" commands, printed inside single quotes'`.)