BlakeWilliams / Elixir-Slack

Slack real time messaging and web API client in Elixir
MIT License
675 stars 182 forks source link

193 Upgrade Poison to version 4.0.1 #204

Closed soriyath closed 4 years ago

soriyath commented 4 years ago

I've tried solving this issue: https://github.com/BlakeWilliams/Elixir-Slack/issues/193

soriyath commented 4 years ago

The failed check is because Poison v4.0.1 requires Elixir 1.6.0 at least, and the failed check runs on Elixir 1.4.x

Elixir 1.4.0
install.rebar
1.18s$ mix local.rebar --force
install.hex
0.88s$ mix local.hex --force
install.deps
1.67s$ mix deps.get
14.87s$ mix test
===> Compiling parse_trans
===> Compiling websocket_client
===> Compiling mimerl
===> Compiling metrics
==> bunt
Compiling 2 files (.ex)
Generated bunt app
===> Compiling unicode_util_compat
===> Compiling idna
===> Compiling ranch
==> poison
warning: the dependency :poison requires Elixir "~> 1.6" but you are running on v1.4.0
Compiling 4 files (.ex)
== Compilation error on file lib/poison/parser.ex ==
** (CompileError) lib/poison/parser.ex:276: undefined function defguardp/1
could not compile dependency :poison, "mix compile" failed. You can recompile this dependency with "mix deps.compile poison", update it with "mix deps.update poison" or clean it with "mix deps.clean poison"
The command "mix test" exited with 1.
Done. Your build exited with 1.
BlakeWilliams commented 4 years ago

Thanks for tackling this! 👾