-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Logic initialisation + TG bot addition (#3) #5
Conversation
ping api finals + partial service impl NOT TESTED
not tested)
get_fetcher impl done scheduler impl done todo: upd timing on fail, impl for alerts, tg bot as an endpoint
✅ Result of Pytest Coverage---------- coverage: platform linux, python 3.10.11-final-0 ----------
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Пробежался по коду, очень много комментариев, так что оставлю в общем формате пунктов, которые надо исправить
- Очень много кода, который остался от предыдущих версий и больше не используется
- async/await используется очень плохо - в тех местах где он замедляет - он есть, а в тех где он нужен по логике - его нет
- Нет документации
- Проект сделан не по шаблону
- Отсутствует докерфайл и прочие плюшки шаблонизации
💩 Code linting failed, use |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ping'а нет, как будет - досмотрю
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
миграции куда то пропали
тесты падают
нет докерфайла для пингера
💩 Code linting failed, use |
💩 Code linting failed, use |
добавил логгинг
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Описание к ПРу
No description provided.