CenterEdge / Yardarm

OpenAPI 3 SDK Generator for C#
Apache License 2.0
46 stars 6 forks source link

Drop default usage of dynamic #185

Closed brantburnett closed 1 year ago

brantburnett commented 1 year ago

Motivation

Dynamic has significant performance penalties and isn't terribly useful. Offering it is causing difficulties with System.Text.Json, which prefers object.

Modifications

Use object instead of dynamic by default. Serializers may still map dynamic types to these objects, some casting will be required by consumers to use them depending on their serializer.

brantburnett commented 1 year ago

/merge