developmentseed / titiler

Build your own Raster dynamic map tile services
https://developmentseed.org/titiler/
MIT License
755 stars 156 forks source link

fastapi-slim replaces fastapi #815

Closed n8sty closed 3 months ago

n8sty commented 3 months ago

What I am changing

Replace the fastapi dependency with fastapi-slim.

See https://github.com/tiangolo/fastapi/pull/11522 for details. My brief summary of what happened in the FastAPI project: Installing fastapi now comes along with dependencies that are helpful for development and deployment, at the expense of installing more dependencies. Since tiler applications are opinionated in their dependencies additional dependencies raise the risk of conflicts, albeit slightly. This change keeps the same dependencies that FastAPI pulls in transitively.

How I did it

Swap the dependency specification, meaning nothing would change :smile:.

How you can test it

Unit tests, standard CI should be more than enough because this result in nothing changing.

Related Issues

N/A.

vincentsarago commented 3 months ago

such a shame, I don't really get why they decided to change this 😬

@n8sty thanks for the PR 🙏