26remph / billing Goto Github PK

Здравствуйте ! Отлично поработали ! Молодец разобрались с poetry и ruff (я и сам использую ruff :) ) Оставил небольшие рекомендации:

1. Здесь не стоит хранить апи ключ в репозитории, из соображений безопасности. То же касается и паролей к БД.
2. Тут можно использовать async_sessionmaker https://docs.sqlalchemy.org/en/20/orm/extensions/asyncio.html
3. Тут некритично, но я бы рекомендовал писать более лаконичные описания ) вроде Database session manger.
4. Тут можно вместо цифровых кодов использовать https://docs.python.org/3/library/http.html#http.HTTPStatus
5. Тут похоже неплохо иметь TODO с планом что нужно доделать.
6. Тут видимо просто ручка - заглушка ?
7. Тут закомментированный код можно убрать.
8. Тут по ЮКасса просто модули заглушки (они просто пустые)?
9. Тут отладочный принт можно убрать.
10. Здесь немного не понял зачем этот класс ?
11. Тут и тут похоже на заглушки - имплементации нет.
12. Тут почему у вас __main__.py с двумя подчеркиваниями, зачем так ? Ведь в коде я встретил у Вас и просто main.py.