https://github.com/Student-Labs-2024/effectiveFoosballApp

https://github.com/Student-Labs-2024/effectiveFoosballArduino

https://github.com/Student-Labs-2024/effectiveFoosballAppMattermostBot

• снова 2 тестовых пакета, которые не используется, говорим каждый спринт → architecture 0

• Работа с Pull Requests не систематизирована и Работа с ветвлением не систематизирована

  • Внесены изменения в README.md напрямую из веб-версии, без веток и PR.

• Отсутствие единообразия в системе именования папок и файлов проекта

  где-то каждое слово с большой буквы

  

где-то все в одно слово

• какие-то комментарии на русском, какие-то на английском https://github.com/Student-Labs-2024/effectiveFoosballApp/blob/develop/app/src/main/java/band/effective/foosball/presentation/components/MarqueeText.kt
• Работа с Pull Requests не систематизирована
  • в боте нет PR
• Работа с ветвлением не систематизирована
  • в боте нет веток, кроме основной
• в Python файлы принято называть через snake case
  • https://en.wikipedia.org/wiki/Snake_case

• Хардкод — 4999? Похоже на цену в Эльдорадо. Второй момент по коду ниже: обязательно ли усыплять поток? И почему именно на столько? Здесь же: комментарии на английском, но в другой части кода имеются на русском.

• Отсутствуют или некорректно настроены правила игнорирования изменений

  • нет файла .gitignore в боте
  • pycache не надо вручную удалять

• Наличие изменений, которые не несут смысла

  • закомментированный код