Move the German addons to gobl.de.xinvoice - #927
Conversation
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #927 +/- ##
==========================================
- Coverage 94.99% 94.97% -0.03%
==========================================
Files 327 324 -3
Lines 17732 17540 -192
==========================================
- Hits 16845 16658 -187
Misses 547 547
+ Partials 340 335 -5 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
|
There was a problem hiding this comment.
Pull request overview
Moves German XRechnung and ZUGFeRD addons from core to the approved external gobl.de.xinvoice module.
Changes:
- Registers both German addon keys as external addons.
- Removes core implementations, tests, generated data, and examples.
- Documents the migration and required imports.
Reviewed changes
Copilot reviewed 13 out of 15 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
CHANGELOG.md |
Documents migration and imports. |
addons/addons.go |
Removes bundled German addon imports. |
addons/external.go |
Registers German external addons. |
addons/de/zugferd/zugferd.go |
Removes core ZUGFeRD implementation. |
addons/de/xrechnung/xrechnung.go |
Removes core XRechnung registration. |
addons/de/xrechnung/bill_invoices.go |
Removes XRechnung invoice rules. |
addons/de/xrechnung/bill_invoices_test.go |
Removes invoice rule tests. |
addons/de/xrechnung/pay_instructions_test.go |
Removes payment rule tests. |
data/addons/de-xrechnung-v3.json |
Removes generated addon data. |
data/addons/de-zugferd-v2.json |
Removes generated addon data. |
data/rules/de-xrechnung.json |
Removes generated rule data. |
examples/de/invoice-de-de-xr.yaml |
Removes domestic XRechnung example. |
examples/de/invoice-de-es-xr.yaml |
Removes cross-border XRechnung example. |
examples/de/out/invoice-de-de-xr.json |
Removes generated example output. |
examples/de/out/invoice-de-es-xr.json |
Removes generated example output. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The German XRechnung and ZUGFeRD addons move out of core into
github.com/invopop/gobl.de.xinvoice, following the pattern already used for Denmark, France, Brazil, Mexico, Portugal and Saudi Arabia.addons/external.gonow lists both keys as approved external addons.Removed:
addons/de/, the generateddata/addons/de-*.jsonanddata/rules/de-xrechnung.json(external addons have no generated files in core — checked against dk-oioubl), and the two XRechnung examples, which move to the new module.data/schemas/tax/addon-list.jsonstill lists both keys, sourced from the external registration.Sequencing:
gobl.ublandgobl.ciino longer import the deleted packages as of their latest releases, so nothing breaks on merge. Consumers that validate documents declaring these keys pick the registration up throughgobl.dev/bundle(PR linked below). Repos that do not bump GOBL are unaffected.Pre-Review Checklist
go generate .to ensure that the Schemas and Regime data are up to date.And if you are part of the org: