Skip to content
This repository has been archived by the owner on Aug 28, 2023. It is now read-only.

Update deps #6

Merged
merged 3 commits into from
Nov 28, 2019
Merged

Update deps #6

merged 3 commits into from
Nov 28, 2019

Conversation

strzibny
Copy link
Contributor

@strzibny strzibny commented Nov 27, 2019

Update meck and poison to be used in newer projects.

I am using Elixir 1.9:

Erlang/OTP 21 [erts-10.3.5.5] [source] [64-bit] [smp:4:4] [ds:4:4:10] [async-threads:1] [hipe]

Elixir 1.9.0 (compiled with Erlang/OTP 21)

Tests pass locally for the new setup.

@maennchen
Copy link
Contributor

Thanks @strzibny. Could you also raise the required elixir version in the mix.exs to at least 1.6 and adapt the travis-ci config accordingly?

@coveralls
Copy link

coveralls commented Nov 28, 2019

Coverage Status

Coverage increased (+0.3%) to 89.744% when pulling d0db1a0 on strzibny:update-deps into f4530bf on jshmrtn:master.

@strzibny
Copy link
Contributor Author

I updated it. I kept only OTP 21 since 22 is not compatible with Elixir 1.6 so the matrix fails.

@maennchen maennchen merged commit 955dda0 into jshmrtn:master Nov 28, 2019
@maennchen
Copy link
Contributor

Thanks a lot! ❤️

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants