Feedback

[WhiteHodok]

Dislaimer ! Это мнение носит субъективный характер ревъюера, оно лишь даёт рекомендации

По репозиторию

• Всё в 1-й бренче
• Отсутствует requirements.txt (или любое другое название файла с фризом либ питона, необходимых для запуска). [Гайд как это сделать] (https://pip.pypa.io/en/stable/cli/pip_freeze/ "Полезновое")

По боту

• Бот прекрасен

[Goldenbarnowl]

Асинхронный бот с машиной состояний на Telebot в тестовом задании Student Labs

По коду

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

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

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

• Не советую хранить большие объемы текстовых данных прям в боте или txt, лучше воспользуйся json или бд

• Тот самый день когда нужно писать комментарии про каждый хендлер наступил! Комментарии где?)

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

По GitHub

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

• requirements.txt нужен (об этом говорилось в фидбеке от Влада)

• Файлы .idea и pycache - лишние. Надо поработать с .gitignore

По боту

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

• Большая молодец, что попробовала telebot после aiogram, можешь больше не пробовать, aiogram удобнее

[!WARNING]

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

[Goldenbarnowl]

Дополнение: ПАПАВСЯ