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

Make reflex-hosting-cli optional, to lower the dependency count on standalone installs #4733

Open
clemlesne opened this issue Feb 3, 2025 · 2 comments
Labels
enhancement Anything you want improved

Comments

@clemlesne
Copy link

Describe the enhancement you want

It would be better as a dependency management perspective to make reflex-hosting-cli optional in the deps.

All Reflex PaaS/SaaS dependencies could go into an hosting optional dependency group, we could install with reflex[hosting].

As a context, as of v0.1.34, reflex-hosting-cli has those dependencies:

  • charset-normalizer
  • httpx
  • platformdirs
  • pydantic
  • pyyaml (not shared with reflex)
  • rich (not shared with reflex)
  • tabulate (not shared with reflex)
  • typer
@clemlesne clemlesne added the enhancement Anything you want improved label Feb 3, 2025
@adhami3310
Copy link
Member

adhami3310 commented Feb 3, 2025

thanks for your report! i believe reflex does use rich, is there a different reason that it wouldn't be shared?

otherwise, we might look into it. a larger project has been taking all basic reflex features into something like reflex-core or reflex[core]. The default install would still most likely contain reflex-hosting-cli though, maybe it could be possible with an opt out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Anything you want improved
Projects
None yet
Development

No branches or pull requests

2 participants