Skip to content

Conversation

cyclotruc
Copy link
Member

@cyclotruc cyclotruc commented Mar 10, 2025

Refactoring

  1. Code Reorganization:

    • Moved the main entry points (ingest, ingest_async) to entrypoint.py
    • Renamed clone_repo to clone for simplicity
    • Relocated data structures to a dedicated ingestion_schema.py module
  2. Improved Type Definitions:

    • Replaced ParsedQuery dataclass with a Pydantic model IngestionQuery
    • Better isolation of configuration types with the CloneConfig class
  3. PyTest Configuration:

    • Added asyncio_default_fixture_loop_scope = "function" for better async test isolation

@cyclotruc cyclotruc requested a review from atyrode March 10, 2025 23:55
@atyrode
Copy link
Contributor

atyrode commented Mar 10, 2025

Yeah :) Okay

@cyclotruc cyclotruc merged commit b098bb4 into main Mar 10, 2025
18 checks passed
@cyclotruc cyclotruc deleted the refactor/parameters branch March 10, 2025 23:57
filipchristiansen pushed a commit that referenced this pull request Mar 13, 2025
filipchristiansen pushed a commit that referenced this pull request Mar 13, 2025
Signed-off-by: Filip Christiansen <22807962+filipchristiansen@users.noreply.github.com>
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.

2 participants