-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
tree: update hint about missing deps #5005
Conversation
"pip": f"pip install 'dvc[{scheme}]'", | ||
"conda": f"conda install -c conda-forge dvc-{scheme}", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could also colorize it a bit like in updater. Just for consistency.
EDIT: ok, updater needs some refactor too. Keeping as is.
"\n" | ||
f"\t{cmd}\n" | ||
"\n" | ||
f"See {link} for more info." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have thought about adding a troubleshooting entry, but doesn't seem worth it, as install
instructions already have extras described.
" 1) Install specific missing dependencies:\n" | ||
" pip install {}\n" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The usefulness of this and all
is questionable and it seems more confusing than helpful. Left only one specific command instead.
Related to iterative#4997 Fixes iterative#4987
Related to #4997
Fixes #4987
β I have followed the Contributing to DVC checklist.
π If this PR requires documentation updates, I have created a separate PR (or issue, at least) in dvc.org and linked it here.
Thank you for the contribution - we'll try to review it as soon as possible. π