-
Notifications
You must be signed in to change notification settings - Fork 350
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
chore: upgrade CI OTP and Elixir versions #623
Conversation
af8f2f6
to
29862cc
Compare
@teamon, do you have any clue what the proper fix for the unit test is? I am not sure what route to take. |
I’m heading to sleep, I’ll take a look tomorrow! |
This looks like some issue with httparrot maybe? |
29862cc
to
3a197ba
Compare
cb6b3b5
to
af99950
Compare
8238abf
to
b96d822
Compare
Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
b96d822
to
5e5c2ca
Compare
Looks like we need to give this another try. @yordis How many versions back do you want to keep in CI? (for Elixir and Erlang/OTP) |
I am trying to maintain (in CI) the latest 2 major versions, so Elixir 15+ and OTP 25+; wait 2-3 months and it will be Elixir 16+ and OTP 26+ |
I'll do up a new PR to update CI I won't necessarily know what needs to be done to fix failures/warnings in new Elixir versions, but it's a start. |
> I am trying to maintain (in CI) the latest 2 major versions, so Elixir 15+ and OTP 25+; wait 2-3 months and it will be Elixir 16+ and OTP 26+ elixir-tesla#623 (comment)
> I am trying to maintain (in CI) the latest 2 major versions, so Elixir 15+ and OTP 25+; wait 2-3 months and it will be Elixir 16+ and OTP 26+ elixir-tesla#623 (comment)
Can this be closed now that #682 is merged? |
I need to follow up on adding |
Signed-off-by: Yordis Prieto yordis.prieto@gmail.com