Skip to content

Conversation

@MateoLostanlen
Copy link
Member

Summary

This PR removes the unused ETL service and simplifies the developer workflow.

Changes

  • Removed ETL

    • Dropped etl_scripts service from docker-compose.yml
    • Removed run-etl and ETL references from Makefile
  • Makefile

    • Added init target for .env creation (run once)
    • Simplified run (frontend + tools) and run-all (frontend + tools + engine)
    • Reordered and cleaned up targets for clarity
  • README

    • Added a Quick Start section with minimal commands:

      make init
      make build
      make run
    • Updated instructions to send alerts via Jupyter notebooks and view them in the frontend or API

    • Documented services, access URLs, and credentials

    • Removed all ETL references

@MateoLostanlen MateoLostanlen requested review from fe51 and m-julio October 1, 2025 12:16
Copy link
Member

@fe51 fe51 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for the PR, find it more clear now.

I think I will do one more iteration of the readme, but for sure, it's already better than before.

I just made a few comments

@MateoLostanlen MateoLostanlen requested a review from fe51 October 1, 2025 14:34
Copy link
Member

@fe51 fe51 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the udpates !
LGTM !

@MateoLostanlen MateoLostanlen merged commit 037768b into main Oct 2, 2025
1 check passed
@MateoLostanlen MateoLostanlen deleted the clean_doc branch December 19, 2025 08:03
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.

3 participants