search

piccolo-orm / piccolo_admin

A powerful web admin for your database.
https://piccolo-orm.com/ecosystem/
MIT License
316 stars 37 forks source link

Swap dependency `fastapi` to `fastapi-slim` #392

Closed metakot closed 1 month ago

metakot commented 4 months ago

So the author of fastapi makes this really unwanted change. See https://github.com/tiangolo/fastapi/pull/11522 Some authors (like this) already did the rollback, maybe we too?

dantownsend commented 4 months ago

I think for Piccolo Admin, using fastapi-slim is sufficient, so makes sense to change it.

metakot commented 1 month ago

The horonable sir of FastAPI did the [standard] optional dependencies. Issue resolved :)

dantownsend commented 1 month ago

Cool, thanks for the update.