Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for Elixir 1.8 #64

Merged
merged 1 commit into from Jun 27, 2019
Merged

Add support for Elixir 1.8 #64

merged 1 commit into from Jun 27, 2019

Conversation

rslota
Copy link
Contributor

@rslota rslota commented Jun 18, 2019

This commit adds support for Elixir 1.8 by:

  • Upgrading Distillery to 2.0
  • Upgrading other, less relevant dependencies that won't compile on
    Elixir 1.8 or generate lots of "deprecated" warnings
  • Small code changes to fix warnings generated by newer versions of
    Elixir
  • Minimal Elixir version of project has changed to 1.6, as it's not
    possible to make everything compile with a range of Elixir 1.4 - 1.8
  • Elixir 1.8 / Erlang 22.0 added to CI build matrix

@coveralls
Copy link

coveralls commented Jun 18, 2019

Coverage Status

Coverage increased (+0.2%) to 90.476% when pulling 2d13a9e on rs/update_distillery into 55784ab on master.

with_mock: 3,
with_mocks: 2
]
import Mock
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are some magic functions required by Mock to be imported while using Mock.with_mock. This means, that the "suggestion" from Mock documentation to import Mock is actually a requirement.

@rslota rslota changed the title [WIP] Update project to work with Elixir 1.8 Update project to work with Elixir 1.8 Jun 19, 2019
@rslota rslota changed the title Update project to work with Elixir 1.8 Add support for Elixir 1.8 Jun 19, 2019
@rslota rslota requested a review from bartekgorny June 19, 2019 07:14
@rslota rslota force-pushed the rs/update_distillery branch 3 times, most recently from 468f6e6 to c2dd531 Compare June 19, 2019 07:30
@rslota rslota requested a review from arkgil June 26, 2019 13:54
@arkgil
Copy link
Contributor

arkgil commented Jun 26, 2019

@rslota I suggest adding a support for Elixir 1.9 in one go 😉

This commit adds support for Elixir 1.9 by:
* Upgrading Distillery to 2.0
* Upgrading other, less relevant dependencies that won't compile on
  Elixir 1.8/1.9 or generate lots of "deprecated" warnings
* Small code changes to fix warnings generated by newer versions of
  Elixir
* Minimal Elixir version of project has changed to 1.6, as it's not
  possible to make everything compile with a range of Elixir 1.4 - 1.9
* Elixir 1.8, 1.9 / Erlang 22.0 added to CI build matrix
@rslota
Copy link
Contributor Author

rslota commented Jun 27, 2019

@rslota I suggest adding a support for Elixir 1.9 in one go 😉

Done!

@arkgil arkgil merged commit 358ba70 into master Jun 27, 2019
@arkgil arkgil deleted the rs/update_distillery branch June 27, 2019 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants