Feedback

[Goldenbarnowl]

По коду

• Файл main.py пустой

• Импорты должны быть сверху файла

• Вставляй пустую строку между каждой группой импортов.

• Импорты должны быть сгруппированы в следующем порядке:

  1.  импорты из стандартной библиотеки
  2.  импорты сторонних библиотек
  3.  импорты модулей текущего проекта

• Названия переменных должны быть логичными, а не набором букв

[!IMPORTANT] Ознакомься с PEP 8 и будь внимательнее

По GitHub

• Нет .gitignore

• README.md очень важен, нужно расписывать полный процесс запуска и описание .env "Многие очевидные вещи - неочевидны"

• requirements.txt - пожалуйста, посмотри как делать этот файл правильно

• Файлы .idea и pycache лишние

По боту

• Инициализация бота должна проходить 1 раз, а не в каждом файле

• По ТЗ должна быть машина состояний. Машина состояний должна быть в каждом боте!

• Все клавиатуры должны быть описаны в отдельном файле

[!WARNING]

• Мнение ревьюера является субъективным и носит рекомендательный характер!!!
• Некоторые пункты могут повторяться у других ревьюеров (являются дополнениями)
• При необходимости Менторы направления Telegram Bot всегда могут устроить созвон в нашем канале Discord и обсудить все тонкости кода. (Писать на аккаунт Student labs)
• Фидбек Ментора не является окончательным перечнем ошибок и недочетов кода (это скорее советы, которые мы посчитали необходимыми)

[WhiteHodok]

Feedback

Всё плохо, но есть и плюсы

• README файл расписан +- верно

Заключение

• Полностью согласен с мнением ревьюера выше(README пойдёт), оно исчерпывающее

• Мнение ревьюера полностью субъективно и носит рекомендательный характер