e-kondr01 / fastapi-sqlalchemy-toolkit

Declarative filtration, database level validation and other features for FastAPI + Async SQLAlchemy applications
MIT License
13 stars 5 forks source link

fixed bug with syntax error on ModelManager class #29

Closed mainStorne closed 1 month ago

mainStorne commented 1 month ago

Синтаксическая ошибка в классе ModelManager. Она возникла из-за квадратныъ скобок в классе без явного указания что это generic. Запуск на python 3.11

e-kondr01 commented 1 month ago

Это не баг, это новый формат параметров типа, который добавлен в Python 3.12:

https://docs.python.org/3/whatsnew/3.12.html#pep-695-type-parameter-syntax

Но я согласен, что для обратной совместимости с Python < 3.12 можно оставить с Generic, как было ранее. Запушил другой коммит, так как в твоём не был применён isort/ruff к импортам:

https://github.com/e-kondr01/fastapi-sqlalchemy-toolkit/commit/2b1727b7e8663e3c7eb4789dbd0a9104e42df7e2