Skip to content
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

Hide commands #54

Closed
dmpetrov opened this issue Jul 15, 2024 · 5 comments · Fixed by #79
Closed

Hide commands #54

dmpetrov opened this issue Jul 15, 2024 · 5 comments · Fixed by #79
Assignees

Comments

@dmpetrov
Copy link
Member

Our CLI experience is not ready yet. To not to confuse users, we should make the commands invisible (without removing the commands).

-V the only command we should be exposing.

@dmpetrov dmpetrov mentioned this issue Jul 15, 2024
3 tasks
@skshetry
Copy link
Member

@dmpetrov, would you be open to disabling cli completely? -V does not seem very important, you can access that during runtime or with pip show datachain.

We can still make it available as python -m datachain.

@dmpetrov
Copy link
Member Author

Good idea! We should not forget to change the bug report template 🙂

@skshetry
Copy link
Member

skshetry commented Jul 17, 2024

@dmpetrov, what should we do about datachain query? It works as expected AFAIK.

EDIT: show works too, and can be adapted for DataChain with minimal changes.

@dmpetrov
Copy link
Member Author

yes, I know, but let's hide everything for now.

@skshetry
Copy link
Member

Okay, looks like we cannot just disable cli.

[datachain_exec_path, "--internal-run-udf"],

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants