search

BlakeWilliams / Elixir-Slack

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

Fix application list #196

Closed axelson closed 5 years ago

axelson commented 5 years ago

Previously all applications were directly specified, however, :poison was missed. This means that if you create a release, the :posion would not exist. Instead switch from :applications to :extra_applications so that Elixir can build the list for us.

Requires Elixir 1.4 which was released 2.5 years ago: https://elixir-lang.org/blog/2017/01/05/elixir-v1-4-0-released/

Fixes #195

BlakeWilliams commented 5 years ago

@axelson Thanks for this! It looks like tests are failing due to travis still trying to test against Elixir 1.3. Can you update the travis config so I can merge this?

axelson commented 5 years ago

Okay updated .travis.yml to remove 1.3 and add 1.6.6 and 1.9.1.

axelson commented 5 years ago

Ping :)

BlakeWilliams commented 5 years ago

Pong!

Sorry, this slipped past my TODO list. I'll get this in and cut a release shortly (likely today).

axelson commented 5 years ago

Great, thanks! :)

BlakeWilliams commented 5 years ago

Just published 0.20.0, thanks for contributing!

axelson commented 5 years ago

Thanks for merging!