search

zefchain / serde-reflection

Rust libraries and tools to help with interoperability and testing of serialization formats based on Serde.
Apache License 2.0
137 stars 26 forks source link

Upstreams Aptos Labs patches #34

Open gedigi opened 1 year ago

gedigi commented 1 year ago

Summary

These changes were made by the Aptos Labs team from a fork of @novifinancial/serde-reflection. Pushing them upstream to remove dependency on forked version.

gedigi commented 1 year ago

@ma2bd I'm planning to open this PR but I'm having issues with local testing, mostly reverse-engineering the packages to be installed on the system, and the fact that the .NET versions supported in serde-generate are out of support from Microsoft. Any tips on how to setup a local test environment? Also, should I provide a patch that targets .NET 7, the currently supported version? Thanks!

ma2bd commented 1 year ago

@gedigi Thanks for sharing your changes upstream. I think we could update the version of .NET in CI and officially in this repo.