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

How to set a default typer command? #25

Open
JohnGiorgi opened this issue May 13, 2021 · 2 comments · May be fixed by #66
Open

How to set a default typer command? #25

JohnGiorgi opened this issue May 13, 2021 · 2 comments · May be fixed by #66
Assignees
Labels
invalid This doesn't seem right

Comments

@JohnGiorgi
Copy link
Owner

Because of the nesting of our typer apps, we end up with a weird situation where one has to specify a final command, when intuitively we would like a default. E.g. seq2rel-ds preprocess <command> main. Ideally, we would like to be able to drop main, with the assumption that this is the subcommand we want if no other subcommand is specified. I have tried to figure out how to to this with typer with no luck. Keep an eye on this issue in case it is ever resolved.

@JohnGiorgi JohnGiorgi self-assigned this May 13, 2021
@JohnGiorgi JohnGiorgi added the invalid This doesn't seem right label May 13, 2021
@mr-rodgers
Copy link

mr-rodgers commented Dec 27, 2021

Has anybody ever found a solution to this? The linked issue is already closed, and the solution proposed in there doesn't actually work for this specific use case.

It yields the exact same problem as in this comment

@JohnGiorgi
Copy link
Owner Author

Hi @mr-rodgers

I have not found a specific fix for this. You might have more luck commenting on this issue here: fastapi/typer#18, even though it's closed.

@JohnGiorgi JohnGiorgi linked a pull request Apr 3, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants