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

Improve for the CLI #267

Closed
B4nan opened this issue Jul 4, 2024 · 0 comments · Fixed by #271
Closed

Improve for the CLI #267

B4nan opened this issue Jul 4, 2024 · 0 comments · Fixed by #271
Assignees
Labels
high priority Do this ASAP! This is for mission-critical work or work that blocks other teams in their work. t-tooling Issues with this label are in the ownership of the tooling team.

Comments

@B4nan
Copy link
Member

B4nan commented Jul 4, 2024

When running pipx run crawlee it tries to create a project, while it should print a help message showing what commands are available, something like this:

image

pipx run crawlee --version/-V is not working at all

image

pipx run crawlee create foo fails hard when the directory already exists

image

Ideally, we should check this early and prompt the user for a different folder name (interactively), this needs to be cleared before we ask the user to pick the template.


There is no success message after the template is created.

image

Should be something like the JS version is doing:

image
@B4nan B4nan added t-tooling Issues with this label are in the ownership of the tooling team. high priority Do this ASAP! This is for mission-critical work or work that blocks other teams in their work. labels Jul 4, 2024
@janbuchar janbuchar self-assigned this Jul 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high priority Do this ASAP! This is for mission-critical work or work that blocks other teams in their work. t-tooling Issues with this label are in the ownership of the tooling team.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants