chore: archiveer eenmalige data-scripts in data/archive/ #73

Closed
opened 2026-04-12 16:10:25 +02:00 by Ghost · 1 comment
Ghost commented 2026-04-12 16:10:25 +02:00 (Migrated from codeberg.org)

Probleem

De data/-map bevat 14+ scripts, deels legacy. Een nieuwe bijdrager (of toekomstige ik) weet niet welke scripts actief zijn en welke eenmalig waren.

Scripts die niet meer actief zijn:

  • fetch-coastlines.js — voor een aanpak die niet meer werkt (zie project_osm_pipeline.md memory)
  • process-coastlines.js — idem
  • fetch-schelde-relations.js — eenmalig uitgevoerd

Oplossing

Verplaats eenmalige scripts naar data/archive/:

mkdir data/archive
git mv data/fetch-coastlines.js data/archive/
git mv data/process-coastlines.js data/archive/
git mv data/fetch-schelde-relations.js data/archive/

Actieve pipeline blijft in data/:

  • fetch-overpass.js — Overpass API ophalen
  • process-overpass.js — chain + RDP + output
  • build-polygons.js — polygonen samenvoegen
  • process-gewesten.js — Belgische gewesten

Voeg een korte README toe in data/ die de pipeline beschrijft.

Impact

  • Hygiëne, geen functionele wijziging
  • Makkelijker onboarding voor nieuwe bijdragers

Risico

Laag — alleen bestandsverplaatsingen.

Zie REFACTORING.md voorstel #9

## Probleem De `data/`-map bevat 14+ scripts, deels legacy. Een nieuwe bijdrager (of toekomstige ik) weet niet welke scripts actief zijn en welke eenmalig waren. Scripts die **niet meer actief** zijn: - `fetch-coastlines.js` — voor een aanpak die niet meer werkt (zie project_osm_pipeline.md memory) - `process-coastlines.js` — idem - `fetch-schelde-relations.js` — eenmalig uitgevoerd ## Oplossing Verplaats eenmalige scripts naar `data/archive/`: ```bash mkdir data/archive git mv data/fetch-coastlines.js data/archive/ git mv data/process-coastlines.js data/archive/ git mv data/fetch-schelde-relations.js data/archive/ ``` Actieve pipeline blijft in `data/`: - `fetch-overpass.js` — Overpass API ophalen - `process-overpass.js` — chain + RDP + output - `build-polygons.js` — polygonen samenvoegen - `process-gewesten.js` — Belgische gewesten Voeg een korte README toe in `data/` die de pipeline beschrijft. ## Impact - Hygiëne, geen functionele wijziging - Makkelijker onboarding voor nieuwe bijdragers ## Risico Laag — alleen bestandsverplaatsingen. _Zie REFACTORING.md voorstel #9_
Ghost commented 2026-04-12 21:20:58 +02:00 (Migrated from codeberg.org)

Geïmplementeerd in refactor-commit 96e1a18.

Geïmplementeerd in refactor-commit 96e1a18.
Sign in to join this conversation.
No milestone
No project
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
jelmer/topoquiz#73
No description provided.