-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for .Net Core #54
Comments
Quite interested. I just have time restrictions. There are lots of other enhancements apart from support for .NET Core in my mind that I simply don't have the time to do. |
Okay, Sina, I gave it a shot for a full day. So far I have achieved the following:
Looks like tests will become kind of a headache for 3 reasons:
Got under .Net Core much more properties serialized (same with e.g. PathsExampleTest):
I uploaded the current (unbeautified) state of my YAXLib fork to https://github.com/axunonb/YAXLib |
Hi Sina, the first port to .Net Core is completed and uploaded to GitHub (https://github.com/axunonb/YAXLib). All 216 unit test for .Net Framework and the 200 unit test relevant for .Net Core run successfully, |
👍 🥂 Impressive! |
Do you got it running on appveyor? Also isn't it better to move to one SLN and project file? Or maybe that is a good next step. |
Well, I had my ups and downs with this port :) but still: cheers. |
@sinairv this is a major feature! |
I'll start to integrate it into a project now, as first proof of concept. Keep fingers crossed. |
I'm super excited. Lots of thanks @axunonb. This is great work. I will look into it in more details tomorrow. Thanks again. |
Thanks, Sina, you're welcome. I worked a bit more on remaining tests for .Net Core. Now Collections and Dictionaries are included, but I replaced
Eventually |
Have you ever considered adding support for .Net Core? With the current version of .Net Core XML serialization is quite bulky and having YAXLib at hand would be a relief. I already gave it a try and there would of course be some code adjustments necessary, mainly for conditional compilation in terms of reflection. But by making use of a modified version of the extension methods from https://github.com/StefH/ReflectionBridge it should be feasible. Interested?
The text was updated successfully, but these errors were encountered: