-
Notifications
You must be signed in to change notification settings - Fork 49
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
Replace Poison with Jason #32
base: master
Are you sure you want to change the base?
Conversation
What is the motivation for using |
Jason is faster and "smaller", many projects already switched to Jason. |
I came here to request these changes as well. As Phoenix comes with Jason already as a dependency, many projects are following suite to keep from bringing in dependencies that do the same thing. If you don't want to use a json library directly, then maybe we could tie it to a behaviour or something like https://github.com/scrogson/oauth2/#configure-a-serializer Another example of people asking for this |
I'm not opposed to using Jason :) Mostly I need to figure out why Travis isn't running the CI jobs anymore :( |
Can somebody merge this? Oh, I see Poison has a new maintainer now 🤔 |
Poison
withJason
for JSON encoding/decodingmix format