sneako / finch

Elixir HTTP client, focused on performance
MIT License
1.23k stars 114 forks source link

Fix calls to deprecated Logger.warn/2 #232

Closed solar05 closed 1 year ago

solar05 commented 1 year ago

I'm not sure if this is the right way. Maybe it's also possible to bump the minimum Elixir version to 1.11? But I guess that would be a breaking change :/

wojtekmach commented 1 year ago

+1 for requiring Elixir 1.11.

wojtekmach commented 1 year ago

If we merge this as is, we should update Elixir requirement in mix.exs.

solar05 commented 1 year ago

Done, also bump elixir CI version 👌

sneako commented 1 year ago

Thank you!