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

• Отсутствует пустая строка в конце файла
  • https://github.com/Student-Labs-2024/effectiveFoosballApp/pull/1/files#
    • app/src/main/res/values/strings.xml
  • https://github.com/Student-Labs-2024/effectiveFoosballApp/pull/2/files
    • app/src/main/java/com/example/effectivefoosball/ui/theme/Type.kt
    • и т.д.
  • Рекомендуется настроить линтер (Не настроен линтер)
• Работа с коммитами не систематизирована
  • есть на русском и на английском комментарии
• Хардкод
  • текстовые константы обычно размещают в файле res/strings.xml
    • app/src/main/java/com/example/effectivefoosball/presentation/components/NavigationPanel.kt
• Работа с Pull Requests не систематизирована
  • Не все изменения идут через PR (readme в main)
  • В описании PR нет инфо об номерах тасок
  • PR огромны
• Не нашли репо с прошивкой и ботом, при этом заявлено, что работа уже идёт
• Работа с ветвлением не систематизирована
  • за Branching балла не будет, поменяли README только в ветке main вручную через web интерфейс на Github скорее всего. Актуальных изменений этого файла нет в других ветках, где ведется работа
• комментарии в коде на разных языках (проверяется весь репозиторий)
• Отсутствие / неправильное описание репозитория
  • за Documentation будет балл, но говоря об инструкции в README md имелось в виду инструкция по сборке приложения для разработчиков :)
• Есть две директории под тесты, но при этом они складываются в main пакет.
• рекомендуем сделать правильное имя пакета, сейчас пакет называется com.example.effectivefoosball; можно сделать band.effective.foosball

Оценка: 2/5.