search

Ezereul / ugc_sprint_1

Спринт 8. Репозиторий для командного проекта модуля UGC
0 stars 0 forks source link

Code review #15

Open BigDeepBlue opened 5 months ago

BigDeepBlue commented 5 months ago

Отличная работа! ETL - 🔥

  1. Не смог найти разницу между etl/src/components/consumer.py и etl/src/consumer.py. Они точно нужны оба?
  2. Кстати, BaseSettings - вполне независимый и может работать не талько в FastAPI, в API текущего спринта тоже можно использовать в flask_app/config/settings.py
  3. Давайте сразу подготовим API к расширению и вынесем сервисный слой, а то сейчас вся бизнес-логика в эндпоинтах.
  4. API без тестов не может долго существовать. Предлагаю сделать пару-тройку.
BigDeepBlue commented 5 months ago

LGTM