02.08.2024 | Notion

Untitled

токен следует хранить в переменных окружения, ни в коем случае его нельзя хранить в репозитории; url - в переменной окружения

Не настроен линтер

весь функциональный код бота находится в одном файле; так быть не должно → декомпозируйте
Наличие изменений, которые не несут смысла из-за Отсутствуют или некорректно настроены правила игнорирования изменений
- если папка .idea в репозитории еще тема для обсуждений, то игнорирование файлов .pyc (pycache) - база
непонятно, как собирался Docker-образ asavovasav/getting-started; и название странное; и в документации указано, что используется Kotlin, хотя бот на Python
для разработки драйвера для Mattermost рекомендуем вместо Flask использовать FastAPI
для определения моделей рекомендуется использовать pydantic, либо dataclasses, если не хотите тянуть зависимости; но в виду предыдущего комментария подходящим решением будет pydantic;
Работа с коммитами не систематизирована
- в боте коммиты не по convential commits
- в Ардуино тоже
- в приложении тоже
Работа с Pull Requests не систематизирована
- в боте нет PR
- в Ардуино тоже
- нет единой структуры названий
- нет описания
Работа с ветвлением не систематизирована
- в боте одна долгоживущая ветка, не используется никакая методология работы с Git
- в Ардуино тоже
все еще 2 пакета с тестами, но тестов там нет
- https://github.com/Student-Labs-2024/effectiveFoosballApp/tree/develop/app/src
Хардкод

Untitled

комментарий от Android-разработчиков Effective:
- тут возможна утечка памяти в случае, если вью была деинициализирована, а вью модель может быть не собрана сборщиком мусора, т. к. она захвачена UsbManager’ом: https://github.com/Student-Labs-2024/effectiveFoosballApp/blob/develop/app/src/main/java/band/effective/foosball/arduinoConnection/ArduinoUsbManager.kt#L24