Skip to content

update v14.2#12

Open
NikolaMandic wants to merge 1 commit intomururu:masterfrom
NikolaMandic:master
Open

update v14.2#12
NikolaMandic wants to merge 1 commit intomururu:masterfrom
NikolaMandic:master

Conversation

@NikolaMandic
Copy link
Contributor

small changes to keep it up to date with the latest version

@lsimoneau
Copy link

I just checked and this branch still passes all the tests on the latest Elixir. The build is failing because .travis.yml has an outdated Erlang version. If you wanted to update this PR to have:

otp_release:
  - 17.1

Maybe @mururu would be able to merge then? Would be great to have this working on current Elixir.

@NikolaMandic
Copy link
Contributor Author

what do you mean with to update pull request to have otp_release -17.1?
Do you mean to somehow specify erlang version?
Since now Elixir is v15 it is not latest now

@lsimoneau
Copy link

Sorry if I was unclear. The Travis CI tests are erroring because Travis is configured to install Erlang R16B, which is incompatible with the latest Elixir. You don't need to specify an Erlang version for the app, it's just for the CI tests.

If you update the .travis.yml file as I mentioned above this build will pass, and hopefully @mururu can merge your branch.

@lsimoneau
Copy link

I've tested your branch on Elixir 15.0 and it works fine, so no need to worry about that.

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.

2 participants