dernbu/lawnet-scraper
1
0
Fork 0
Code Issues 8 Pull requests Projects Releases Packages Wiki Activity Actions

[1/4] Scaffold data-ingest service #12

Merged
dernbu merged 1 commit from feat/1-data-ingest-scaffold into main 2026-06-13 20:09:30 +02:00
Conversation 2 Commits 1 Files changed 9 +251
dernbu commented 2026-06-13 19:44:29 +02:00
Owner
Copy link

Summary

  • add the independent Python 3.13/uv package root and lockfile
  • add a minimal Hello World entry point for the future ingestion service
  • use a straightforward single-stage, non-root Docker image
  • add one focused CLI test
  • add repository ignore rules before later service scaffolds

Size

251 added lines, including the generated 141-line uv.lock.

Verification

  • uv run ruff format --check .
  • uv run ruff check .
  • uv run pyright
  • uv run pytest (1 passed)
  • docker build .

Stack 1 of 4. Merge this PR first.

## Summary - add the independent Python 3.13/uv package root and lockfile - add a minimal Hello World entry point for the future ingestion service - use a straightforward single-stage, non-root Docker image - add one focused CLI test - add repository ignore rules before later service scaffolds ## Size 251 added lines, including the generated 141-line `uv.lock`. ## Verification - `uv run ruff format --check .` - `uv run ruff check .` - `uv run pyright` - `uv run pytest` (1 passed) - `docker build .` Stack 1 of 4. Merge this PR first.
dernbu reviewed 2026-06-13 19:53:55 +02:00
services/data-ingest/Dockerfile Outdated
@ -0,0 +1,33 @@
FROM python:3.13.9-slim-bookworm AS builder
dernbu commented 2026-06-13 19:52:58 +02:00
Author
Owner
Copy link

I feel we dont need builder runtime image stages here. Dont need this optimkzation

I feel we dont need builder runtime image stages here. Dont need this optimkzation
dernbu marked this conversation as resolved
services/data-ingest/src/lawnet_ingest/cli.py Outdated
@ -0,0 +1,50 @@
import argparse
dernbu commented 2026-06-13 19:53:41 +02:00
Author
Owner
Copy link

Lets keep this service empty first, only a print hello world

Lets keep this service empty first, only a print hello world
dernbu marked this conversation as resolved
dernbu force-pushed feat/1-data-ingest-scaffold from cd16b8c361 to 2021dae2d1 2026-06-13 20:05:47 +02:00 Compare
dernbu commented 2026-06-13 20:06:57 +02:00
Author
Owner
Copy link

Addressed both review comments in 2021dae2d127:

  • Simplified services/data-ingest/Dockerfile to a single stage.
  • Reduced the data-ingest service to a Hello World-only CLI and removed the database/egress runtime logic and dependencies.
  • Kept database and TLS connectivity probes in the final wiring PR (#15), where they belong as infrastructure checks.

Verification passed: Ruff format/check, Pyright, pytest (1 test), and Docker build. The stacked branches have been rebased and repushed.

Addressed both review comments in `2021dae2d127`: - Simplified `services/data-ingest/Dockerfile` to a single stage. - Reduced the data-ingest service to a Hello World-only CLI and removed the database/egress runtime logic and dependencies. - Kept database and TLS connectivity probes in the final wiring PR (#15), where they belong as infrastructure checks. Verification passed: Ruff format/check, Pyright, pytest (1 test), and Docker build. The stacked branches have been rebased and repushed.
dernbu merged commit 2021dae2d1 into main 2026-06-13 20:09:30 +02:00
dernbu deleted branch feat/1-data-ingest-scaffold 2026-06-13 20:09:30 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
dernbu/lawnet-scraper!12
No description provided.