-
Notifications
You must be signed in to change notification settings - Fork 62
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
Kotlinx serialization support #179
base: master
Are you sure you want to change the base?
Conversation
I support the changes, but I also understand how unpleasant it will be to add a new library to the standard distribution. From the user's point of view, the solution proposed by @makeevrserg is the best - the presence of an officially supported solution from the kotlin team. But I understand that this will be difficult for the Kotlin team - maybe then make a separate library for this and link to it? |
Is this pr still in progress? Noticed that it has been 7 months without update. And, is it possible to provide a well configured Serializers module so users can avoid adding |
@xiaozhikang0916 The PR is not in draft - it's finished. I didn't updated it because there's no answer from team. I personally don't like |
1f61892
to
4ae842c
Compare
This PR contains serializers along with tests for them:
ImmutableListSerializer
PersistentListSerializer
ImmutableSetSerializer
PersistentSetSerializer
PersistentHashSetSerializer
ImmutableMapSerializer
PersistentMapSerializer
PersistentHashMapSerializer
This PR is relevant to #63