Co vs CO

[avaldebe]

closes #42

Not sure how to address this issue.
My first try was to exclude from the CLI thge pollutants that would confuse typer.
Currently, I use click to construct the sub commands.

[avaldebe]

please note that the first commit on this PR is the same as #44

[JohnPaton]

Does it even make sense to keep typer as a dependency with this change? Seems like we're entirely working around it and essentially just using click (which is also totally fine, I like click too)

[avaldebe]

Does it even make sense to keep typer as a dependency with this change? Seems like we're entirely working around it and essentially just using click (which is also totally fine, I like click too)

Writing a CLI with typer is much easier than with click, but I see your point.
It really depends if typer fixes the problem, see fastapi/typer/discussions/570.
The fix was relatively simple so I submitted a PR fastapi/typer#571.

We can wait for the typer PR to be merged or move to click now, please let me know what you prefer and I'll deal with it.

[avaldebe]

We can wait for the typer PR to be merged or move to click now, please let me know what you prefer and I'll deal with it.

Now that I think more about it, I'll implement the move to pure click while I wait for your answer. I do not mind to revert to typer after they fix the problem, we can keep using click if you are happy with the pure click implementation.

[JohnPaton]

I +1'd your PR, hopefully it makes it in!

[JohnPaton]

The click changes look good, I'm actually surprised how similar they are (though agreed typer is easier when it works)

[avaldebe]

Thanks for approving the PR. I will prepare a new tag/release...

[JohnPaton]

Thank you!