search

fal-ai / fal

⚡ Fastest way to serve open source ML models to millions
https://fal.ai/docs
Apache License 2.0
509 stars 44 forks source link

feat(cli): use rich_click for nicer help #133

Closed efiop closed 6 months ago

efiop commented 6 months ago

We could switch to typer, but for now going with least resistance and giving https://github.com/ewels/rich-click a try.

We already use rich in multiple places, so this is just making ui a little bit more coherent.

Screenshot 2024-03-21 at 02 51 09 Screenshot 2024-03-21 at 02 52 43
linear[bot] commented 6 months ago

FEA-2133 prettify basic cli help

squat commented 6 months ago

btw, I don't love linking issues in private issue trackers to OSS projects/PRs since community has no idea what evil plans are being implemented. Maybe we can track these as basic GitHub issues

efiop commented 6 months ago

Sounds cool! Does merging this change any functionality?

None that I'm aware of. Good point though, let me switch feat to something else maybe... EDIT: looks like it still qualifies as feat from what I understand.

btw, I don't love linking issues in private issue trackers to OSS projects/PRs since community has no idea what evil plans are being implemented. Maybe we can track these as basic GitHub issues

Agreed. Just getting used to linear thing, so overusing it a bit too much right now. Will be going with normal gh issues here from now on.