microsoft / yardl

Tooling for streaming instrument data
https://microsoft.github.io/yardl/
MIT License
30 stars 5 forks source link

Fix codegen for unused unions #91

Closed johnstairs closed 11 months ago

johnstairs commented 11 months ago

The NDJSON codegen would only generate serializers for unions that were used by a protocol. However, we generated serializers for all records, regardless of whether they were used by a protocol. So this would result in complication failures.

The fix is to generate converters for all unions in the model.