edgedb / edgedb-net

The official .NET client library for EdgeDB
https://edgedb.com
Apache License 2.0
82 stars 9 forks source link

Add support for wrapping types in deserialization #63

Closed quinchs closed 1 year ago

quinchs commented 1 year ago

Summary

This PR adds another step in the deserialization pipeline which support wrapping types, ex: Nullable<T>, Option<T>, etc. It also fixes a bug with value type deserialization due to expression initialization by adding an explicit box operation.

scotttrinh commented 1 year ago

Code makes sense to me! Is there any easy test case we can add at least for Nullable<T> to exercise this behavior?

quinchs commented 1 year ago

Added!