Fix group_by_keys/1 for Elixir 1.3

edgurgel / httparrot

HTTP Request & Response Server. An incomplete clone of http://httpbin.org

https://httparrot.onrender.com/

MIT License

86 stars 27 forks source link

Fix group_by_keys/1 for Elixir 1.3 #17

Closed bratsche closed 8 years ago

bratsche commented 8 years ago

Enum.group_by/3 is deprecated now, and it seems to be breaking some tests. This fixes the failing doctest and the other failing test.

edgurgel commented 8 years ago

The tests are failing because Erlang 17 does not accept dynamic keys for maps I think. We can probably drop support for it on travis and bump to Erlang 18+

bratsche commented 8 years ago

It turned out also that Elixir 1.0.2 was not compatible with it, so I bumped Erlang to 18 and Elixir to 1.3.0.

edgurgel commented 8 years ago

Thank you!