search

elixir-tesla / tesla

The flexible HTTP client library for Elixir, with support for middleware and multiple adapters.
MIT License
2.01k stars 348 forks source link

chore: upgrade CI OTP and Elixir versions #623

Closed yordis closed 5 months ago

yordis commented 1 year ago

Signed-off-by: Yordis Prieto yordis.prieto@gmail.com

yordis commented 1 year ago

@teamon, do you have any clue what the proper fix for the unit test is? I am not sure what route to take.

teamon commented 1 year ago

I’m heading to sleep, I’ll take a look tomorrow!

teamon commented 1 year ago

This looks like some issue with httparrot maybe?

nathany-copia commented 5 months ago

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)

yordis commented 5 months ago

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+

nathany-copia commented 5 months ago

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.

nathany-copia commented 5 months ago

Can this be closed now that #682 is merged?

yordis commented 5 months ago

I need to follow up on adding .tool-versions, or please add it yourself 🙏🏻