-
Notifications
You must be signed in to change notification settings - Fork 43
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 kotlinx.serialization #89
Comments
I would be very receptive to PRs, but have no experience with kotlinx serialization. |
@zettelmj Have you been able to make progress on adding Kotlinx Serialization to the library? |
@zettelmj and @ulrikandersen, I am interested in tackling this feature. How has your experience been so far, and do you have any advice for getting started? Thank you for your work so far! |
Hi Sorry for the late answer. I did try to tackle the whole problem at once. In retrospective that is most probably a step too big. I should have gone for the model part first. I would suggest the same: Add the code that allows for the model part of the process in first. I am planning to look into adding end 2 end tests for the existing Jackson based serialisation over the weekend. |
With the release of Take it for a spin, and please do not hesitate to share feedback and/or open PRs with suggestions for improvements. |
This library looks great, are you planning to add support for kotlinx serialization, or would you accept a pr that adds it?
The text was updated successfully, but these errors were encountered: