fastapi / full-stack-fastapi-template

Full stack, modern web application template. Using FastAPI, React, SQLModel, PostgreSQL, Docker, GitHub Actions, automatic HTTPS and more.
MIT License
27.1k stars 4.82k forks source link

💡 Remove unnecessary comment #1260

Closed sebhani closed 1 month ago

sebhani commented 3 months ago

engine is created using create_engine function in the same file (i.e., db.py). Therefore, uncommenting from app.core.engine import engine line is unnecessary and would result in ModuleNotFoundError: No module named 'app.core.engine' error.

tiangolo commented 1 month ago

Cool, thanks!