search

vt-elixir / ja_serializer

JSONAPI.org Serialization in Elixir.
Other
640 stars 148 forks source link

Remove Poison dependency #306

Closed lasseebert closed 5 years ago

lasseebert commented 5 years ago

This is perhaps related to #303

I know that Poison is not used in JaSerializer, only in tests, but it is still a dependency in mix.exs. I guess it could be downgraded to a dependency with something like only: [:test]?

I'm not depending on Poison from any other deps in my own project, but Poison is deployed to production anyway, which is my motivation for this issue :)

I will be happy to provide a PR with an upgraded README, that says something about installing Poison or Jason if not rendering json through Phoenix or Plug.