Skip to content

refactor: saca el venv, arregla deps faltantes, dedup y tests/CI - #1

Merged
AlejandroVegaFullstackDev merged 1 commit into
mainfrom
mejora/cleanup-deps-tests-ci
Jun 13, 2026
Merged

AlejandroVegaFullstackDev merged 1 commit into
mainfrom
mejora/cleanup-deps-tests-ci

Conversation

@AlejandroVegaFullstackDev

Copy link
Copy Markdown
Owner
  • Elimina del repo el entorno virtual completo (8900+ archivos), los pycache y saca el .env del control de versiones (añade .env.example)
  • requirements.txt: añade pandas y python-dotenv (los usaba main.py pero faltaban, el proyecto no corría tras un install limpio)
  • Quita una GA4Connection() muerta del controller (cargaba credenciales y cliente sin usarse en cada instancia)
  • Unifica los dos DTO duplicados (uno con el typo 'Rerport') en GA4ReportDTO, con alias de compatibilidad
  • Mueve GA4Exception al dominio para desacoplar el caso de uso de la API de Google
  • Amplía el Factory Method con los reportes users y page_views (antes comentados)
  • Limpia main.py (envuelto en main()/main, sin bloques comentados) y elimina archivos vacíos (dao, form_request)
  • Tests unitarios (factory + caso de uso, sin red ni credenciales) + GitHub Actions CI
  • .gitignore real y README profesional (estaba vacío)

- Elimina del repo el entorno virtual completo (8900+ archivos), los __pycache__
  y saca el .env del control de versiones (añade .env.example)
- requirements.txt: añade pandas y python-dotenv (los usaba main.py pero faltaban,
  el proyecto no corría tras un install limpio)
- Quita una GA4Connection() muerta del controller (cargaba credenciales y cliente
  sin usarse en cada instancia)
- Unifica los dos DTO duplicados (uno con el typo 'Rerport') en GA4ReportDTO,
  con alias de compatibilidad
- Mueve GA4Exception al dominio para desacoplar el caso de uso de la API de Google
- Amplía el Factory Method con los reportes users y page_views (antes comentados)
- Limpia main.py (envuelto en main()/__main__, sin bloques comentados) y elimina
  archivos vacíos (dao, form_request)
- Tests unitarios (factory + caso de uso, sin red ni credenciales) + GitHub Actions CI
- .gitignore real y README profesional (estaba vacío)
@AlejandroVegaFullstackDev
AlejandroVegaFullstackDev merged commit ddf5513 into main Jun 13, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant