Skip to content

Remove EUROMOD comparator facts from Belgium targets#80

Open
MaxGhenis wants to merge 1 commit into
mainfrom
codex/belgium-target-packages-primary-source-only
Open

Remove EUROMOD comparator facts from Belgium targets#80
MaxGhenis wants to merge 1 commit into
mainfrom
codex/belgium-target-packages-primary-source-only

Conversation

@MaxGhenis

@MaxGhenis MaxGhenis commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove the JRC/EUROMOD-BE comparator source package that was included in Add Belgium target source packages #72
  • remove the EUROMOD alias, comparator regression test, source artifacts, and bundle coverage rows
  • keep the Belgium target set to the six primary publisher streams from Statbel/SPF Finances/ONSS/ONEM-RVA/NBB

Facts

  • Belgium Ledger coverage is now 587 facts across the six primary-source streams
  • EUROMOD/JRC comparator rows are excluded because Ledger facts should come from primary government/statistical publishers

Verification

  • uv run pytest -q tests/test_belgium_targets.py tests/test_ledger_source_package.py
  • uv run pytest -q tests/test_ledger_bundle.py::test_build_bundle_writes_merged_consumer_contract
  • uv run ruff check
  • git diff --check origin/main...HEAD

Refs #69

@MaxGhenis MaxGhenis force-pushed the codex/belgium-target-packages-primary-source-only branch from 003979e to 840f1da Compare July 3, 2026 16:35
@MaxGhenis MaxGhenis changed the title Add Belgium target source packages Remove EUROMOD comparator facts from Belgium targets Jul 3, 2026
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